Yes, i believe so.
If `generate_version_gen_h()` is called with no arguments, `create_version_h.sh` will re-detect version string, each time make is called.
Why? Because else `create_version_h.sh` will not re-detect version string each time make is called, but simply use the version string from the time cmake was called => regression, version string would not auto-update.


看了也白看


variable name is fixed in next commit.


`audio_error_portaudio` already prints the PortAudio error code and error message.

[This](http://portaudio.com/docs/v19-doxydocs/portaudio_8h_source.html#l00104) is the error code definition in PortAudio. I don't think it maps to normal errnos. I could `return -1` as the audio_write_alsa function does in case of error, would that be better?


Just for the record, both versions include the promise-based API but this one does not include the promise polyfill.


Done


first


I saw it reported here #1528 but we didn't have details on how to recreate it, some of our users reported it and I tried to locally fixed it by avoiding it given our scenario, I didn't ever consider a solution to the problem.
Honestly we haven't really dedicated much time on follow the progress of the latest version of cefsharp but we want to update it soon and we know that there will be many conflicts. Thanks for the info.


I think the logic here will be more clear if the move the "real results check to the backup function".
So that here should simply look like
```
if ($confirmed_deletion)  {
     if ($this->backupResults()) {
        // Run delete query
     } else {
       // Alert results could not be backed up and abort
    }
}
```


@timdream Hey Tim,
I've just tried to test it on my flame, but it doesn't work on nightly-latest build from Feb 23.
I've also checked it on this page and it also doesn't work.
While the Romanian word prediction is working well. I'm compairing with Romanian because we pushed dictionaries almost at the same time.
Is it possible that something wrong with uk dictionary or what could be the reason?

Thanks!


Nit: Inconsistent space after `:`.


`__` on purpose?


just nightwatch/commands is enought no ?


Cool. May I merge it? (some work is required to finish the parameters, but it's easy already)

(Probably some more clean and generic mech is required in the library to make it easier to write tc mechs — maybe as plugins. A good reason to think about it)


> I saw it reported here #1528 but we didn't have details on how to recreate it

See  #1594


Sure it's not a problem at all, but it's dead code. Reasonable? Well it's just as reasonable as those silly memsets right before a full memcpy.


it works


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17381](https://teamcity.mesosphere.io/viewLog.html?buildId=146823&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:07:55


That's what happens when we are in rush. 3.3.0 of course has this change.


idk why my system ui and recents lags alot after sometime , and when i restart systemui and the lag goes O.o weird isnt ?


One last thing: `test_make_user_agent_string`, so if someone wants to test just method X they can just `pytest tests -k X` and be sure to hit it.


I've filed https://github.com/mozilla/wp-web-push/issues/191 to use it also in my plugin.


We should probably have a helper method here:

```ruby
def with_adapter_config_setup
  @previous_adapter = ActiveModelSerializers.config.adapter
  yield
ensure
  ActiveModelSerializers.config.adapter = @previous_adapter
end
```


I use the native cpp switch for constant ints only.  This has some
advantages in the compiler I think.
Otherwise I use if - else - if -else - if else ....
Not sure I can do much better than that, although this does run into
problems when the cases are grater than 256 - the compiler basically craps
itself.
I was entertaining the idea of doing an inline-binary tree for strings
(either on hash or on value) and floats.  Would have to be sure the
"operator <" works the same at run time and compile time.
So for me, having the compiler convert into "if (a<b) ..." for everything
other than ints, and into a switch for constant ints (or a binary tree
too?) would be nice.
Bool does not matter so much - if-else should be fine.

I could put the check in when retyping the ast though - might pick up some
errors.

On Fri, Feb 26, 2016 at 4:30 PM, Simon Krajewski <notifications@github.com>
wrote:

> What do you think about this:
>
> type tcpp_const =
>    | CppInt of int32
>    | CppFloat of string
>    | CppBool of bool
> and tcpp_expr_expr =
>    | CppConst of tcpp_const
>    | CppSwitch of tcppexpr * (tcpp_const list * tcppexpr) list * tcppexpr option
>
> This would automatically avoid String-switches and whatever else might
> come up from weird Dynamic code.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/HaxeFoundation/haxe/commit/9d361c6d14e1f6c004a4f9de0a4b8768afcc40bd#commitcomment-16346853>
> .
>


Just for the record, both versions include the promise-based API but this one does not include the promise polyfill. Actually, there is an issue [related with that](https://github.com/mozilla/localForage/issues/369).


So we keep the wavelength in the input file in AA?


If you add a "bin" folder you will also have to update these projects to include the bin folder in the path to zc.exe:

  D:\git\P\Src\Pc\CommandLine\CommandLine.csproj(63):            <HintPath>$(ExtFolder)\Zing\$(UnifiedPlatform)\zc.exe</HintPath>
  D:\git\P\Src\Pc\Compiler\Compiler.cs(418):            var zcProcessInfo = new System.Diagnostics.ProcessStartInfo(Path.Combine(binPath.FullName, "zc.exe"));
  D:\git\P\Src\Pc\InteractiveCommandLine\InteractiveCommandLine.csproj(63):            <HintPath>$(ExtFolder)\Zing\$(UnifiedPlatform)\zc.exe</HintPath>
  D:\git\P\Tst\Tools\RunPTool\RunPTool.csproj(96):            <HintPath>$(ExtFolder)\Zing\$(UnifiedPlatform)\zc.exe</HintPath>

and any paths to "zinger.exe" like this one:

D:\git\P\Tst\Tools\RunPTool\Program.cs(379):                string zingFilePath = Path.GetFullPath(Path.Combine(testRoot, @"..\Ext\Zing\x86\zinger.exe"));

I suspect you need to get a clean clone otherwise your build would have been broken.


On the other hand, i don't quite understand how

`APP_ASSETS` which is a `IDependency[]`

can be passed into

`gulp.src` which expects `string` paths


这里被下线之后应该停止polling了


IMO, this func is hard to read. How about this:
``` swift
public func fadeInOut(completion: AnimatableCompletion? = nil) {
  CALayer.animate({
    let animation = CABasicAnimation(keyPath: "opacity")
    animation.fromValue = 0
    animation.toValue = 1
    animation.timingFunction = CAMediaTimingFunction(name: kCAMediaTimingFunctionEaseInEaseOut)
    animation.duration = CFTimeInterval(self.duration)
    animation.beginTime = CACurrentMediaTime() + CFTimeInterval(self.delay)
    animation.autoreverses = true
    animation.removedOnCompletion = false
    self.layer.addAnimation(animation, forKey: "fade")
  },
  completion: {
    self.alpha = 0
    completion?()
  })
}
```
See https://github.com/raywenderlich/swift-style-guide#closure-expressions for details.


I’d prefer slightly more descriptive name, e.g. `escapedRight`.

(Also in `nmatch`.)


Excellent! Thank you


I'm sorry, but this I is very difficult for me to configure. Is there a way I can just downloaded to phone please


The parent was not used for the type authorization ?


gfgd


the reason I did not do this, was that you should not need to confirm deletion if there's nothing to delete


Ah, then no problem if you prefer it this way. However, we usually try to avoid usage of impure `/usr/bin/env` -- maybe replace it with ${pkgs.iproute}/bin/ip? On the other hand, at this point `postStart` might looks much prettier:
```
postStart = ```
  ip xfrm policy flush
  ip xfrm state flush
  ipsec --stopnflog
```
But again, if that's what is used by upstream I have no problem with you using it if there is no impure paths.


May I ask why roverbody avionics was removed?


Shouldn't it be `runner.useDaemon = true`? You're just reading the property here.


I don't know, as we didn't discuss it, I left the input file as it was, and just changed the column in the pandas frame.
I don't know which unit people use more in this context...
(I also left the frequence commented out, in case we change our minds and we want to include it again)


Looks good to me


По-моему, если селектор заменить на `.photogallery-image img` код будет выглядеть лучше.


No `IllegalStateException` here? Is this intentional?


* Although this is an horrible and very buggy practice (many existing applications expect `C:` to be mandatory), you can actually install Windows on another drive than `C:`. The [`%SYSTEMDRIVE%`](http://ss64.com/nt/syntax-variables.html) environment variable has been made available for this purpose.
* [`%PROGRAMDATA%`](http://ss64.com/nt/syntax-variables.html) points to `%SYSTEMDRIVE%\ProgramData`


From what I've understood from what Eric told us about constant in DB, TINYINT field are only useful when math operations and sort is required.
Otherwise VARCHAR is enough.

Do we really need TINYINT ?


of course. got distracted while typing...


Makes sense. For the first iteration it doesn't really matter much. The tests are really important actually. Great work BTW !


I'm not sure it makes sense for us to test so many versions.

BTW, the commit message says "incremental", but it's for an up-to-date build here :)


I've checked the style-file loading both via file and by setting the line-width to 10. Its' definitely the style-file.

Back on RT, it was on 3.6. By now it seems to have been upgraded to 3.8. I could invest some time trying to downgrade it again, but since the previous version did the same, I currently don't see the need.


```java
public static Direction[] NESW = { NORTH, EAST, SOUTH, WEST };
```


That's great.

Thank you


```java
public static Direction[] WNES = { WEST, NORTH, EAST, SOUTH };
```


И несовсем понятно, зачем два цикла, когда можно было обойтись одним.


it would have been shorter if split result was "streamable". But adding the ceremony of converting using t-rex^HĤ spliterators ruined the idea !
added a trim though


I see possible NPE when last_complete_build is not yet set (before polling succeeds or package has no build at all)


No space after `switch`?


#15 対応しました！


#15 対応しました！！


Spreading knowledge is a very good thing. I bet only the 10% of the ppl who ate already broccoli, probably never knew it belongs to the cabbage family.


Would that really hurt, we are going to throw an error anyway, if it is an NPE we will still know what it is.


Having the `return` here instead of in a `default` is inconsistent with the rest of the code you added; personally I prefer it like this


Figured it out – it was my foolishness with `void *`. `void *` - not even once.


no doc alignment pls


I think, latest laravel needs `web` as middleware, thus it becomes important somewhere to notify, cause, i struggles almost 22 hours until i created dummy application and asked @ #laravel community in irc about issue.

So, this should be by default, if in case, people don't need them, they can remove the unwanted `auth`.


This is just an introduction message


Ah i see, than, its fine in `bind`/`unbind`. I will have look into this bug later.


My thinking is that once you have CppSwitch, it should generate a native `switch` no matter what. If you only use it for Int the case type could even be changed to `(int list * tcppexpr)`. Then when translating TSwitch to the C++ AST you either get CppSwitch (if every case expression is a const int) or some CppIf chain.

I'm interested in a target-agnostic solution for String switches though. I know @waneck has that problem on Java and put some thought into it, and @ncannasse's HL would profit as well if this was optimized properly.


- renamed e1 to e (as in Entry)
- extracted predicates both for valid and invalid messages to more safely partition the set


I could comment on each line!


Add wavelet denoise CL kernel & tuning handler


redundant whitespace


Is ``_slug_is_available`` supposed to be private? If you need to call it in order to validate a slug should it be made public?


aaaa


Can you remove some of the superfluous parts of some of these local variable names? `oppositeDirection` -> `opposite`, `directionComponents` -> `components`, ...


A test is missing to verify that selecting a issue by its key or other fields contains the issue type


And add other comments as well


`int endX = end.getX(), endY = end.getY();`


Please refactor to use a function call here


Primary unit model (unrefined)


Comment on Commit


No test on migrated data ?


The chain of comments is preserved


I thought that we won't do migration of issues, as it will be a too long migration ?


"There is  no item available"


Please fix indentation (I'm not using tab space).


This i.e. is positioned weirdly. Can probably just drop it


Messaging could be reported immediately after operation again just for code clarity. For example
```
$actions = [];
......
.......

$this->dbh->query($delQ);
$actions[] = Deleted columns: $deleted_columns_string;
..........
$this->dbh->query($addQ);
$actions[] = Added columns: $added_columns_string;
.....
.....
.....
// then at the end
if (!$actions) {
     $actions[] = "The results table did not need to be modified.";
}
$this->messages[] += $actions;
```


ack b76e19d :D


ack f2d1c12


このようなエラーがでて、調べたところマルチバイト関数を有効にする=;extension=php_mbstring.dllの;を外すとエラーが治るとの情報があったが、php ini にそもそもextension=php_mbstring.dllが書いてなかったので別のエラーの対処法を知りたい。


Move the blank line down one line instead?


Neat :D


The point is you can write `Splitter.on(',').trimResults().omitEmptyStrings().splitToList(err.getProperty()).stream()`
You can even extract the splitter to make things shorter


ack 8ffe7e8


This should be ```if (len <= 0)```


Even if this fails, I recommend that you try 2.10 on LTC when you find an empty framework.
Also, I guess that 2.05 would pass STC easily because 2.00 and 2.10 are of similar performance.
In such situations I would rely more on such tries that on SPSA. Small changes here are not small changes because this is an important formula! Good luck!


ack 342a03f :D


Ok checked how it's actually used and as long as err ```!= num_bytes``` everything will be fine. so this is not a problem as long as they don't add 983 more errors =)


add some line for my first comment


:fireworks: :tada: :balloon:


usize can never be < 0


Good thing that we could remove this class !


Well, if we have a proper hash function (HL uses Neko one), we just have to switch on the hash then check again the string equality


ack e97c7e4


Neither do I, but I edited this file with the configuration at work, where I have to. Will fix.


No. Some evil forces must have possessed me.
Corrected.


#15 #8


No, I mean if validation fails in line 1255


You could cache if its null:
String name = first == null ? "Null" : first.getClass ().getSimpleName ();

How does that sound? :)


OK, will do.


Do we also need update yamitranscode.cpp?


First, note that I don't have knowledge of how libreswan uses its configuration files. From a little reading, I imagined this:

1. It accepts several connection descriptions, which can be split into several files using `include` option.
2. It can be reloaded from the same file (`ExecReload`).

What are these binary files? Do they have to be alongside configuration files? Does libreswan create them automatically or do you have to specify something? Depending on this we can either try to split configuration and these files, or move it somewhere to `/var/lib` (where mutable state data usually reside) and have NixOS update it as needed.


@jyotisingh very nice xss introduced here. Please escape


You do not yet want to introduce WAReport to simplify the table here?


Why not have used selectParamsByActiveRuleKey ?


And again this morning I received an email at 4:56 and 5:06 with now only the second PHP Notice repeated 18 and 40 times.


Would have been better to use selectParamsByActiveRuleKey, no ?


Ce n'est pas moi qui ait écrit ça..
Et en fait c'est déjà intégré à la branche develop.


合并


cool stuff


En gledens dag!

Tror graphvizen funker mye bedre.
Spesielt når vi begynner å splitte opp i ulike nettverk. Har du tenkte noe på hvilke nettverk vi trenger?
Et public (patron-client), et semi-public (koha-intra, catalinker), ett som kun er internt mellom containerne..

Hadde tenkt å begynne fjerne snakking mellom containerne over internett, og kun bruke direkte dns via nettverk.
Vi kan beholde port-eksponeringene i docker-compose-template-dev, og fjerne de i de andre


Hi Psliwa... After change, I have Issues #9 ... How to fix?


Moreover, doing one SQL query by rule to load parameter is too much, no ?
Would have it been possible to load all params from all rules just before ?


Forget my last point, optimisation can be done later


its mandatory


Just a comment text in the bot.js for testing


would better change to " AVC/HEVC IDR frame interval"


ers


Direct commit to frozen file


Hello @fnichol !

Thanks for using `rpassword`.

Just wanted to let you know, I will simplify the `Cargo.toml` file this weekend, as Cargo now supports things like:
```toml
[target.'cfg(windows)'.dependencies]
winhttp = "0.4.0"

[target.'cfg(unix)'.dependencies]
openssl = "1.0.1"
```

Hopefully, this will make things easier for everyone and we won't have to have all kinds of special architecture dependent config.

Best regards,
Conrad


BTW: yesterday it were 3 emails at 4:56, 5:07 and 5:27, but with the two different notices. All times are UTC.


@reet- The Bitset convertion is a cool idea :) If you like, you can change `to_msi_addr(uint16_t const handle)` to `to_msi_addr(Msi_address::Handle::access_t const handle)`. The Bitset template should choose `uint16_t` automatically according to its arguments.


I might be misunderstanding, but shouldn't this be in the earlier commit?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17382](https://teamcity.mesosphere.io/viewLog.html?buildId=146837&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 238 Build time: 00:09:25


I was seeing temporary strings in the unprotect() in the build I was profiling, then thought it would be much neater.. but then made it _considerably_ less neat with the #ifdefs, so on the whole probably unnecessary since a fully optimised build is fine.


TODO: the hook in `signs` currently only verifies that the next certificate in the chain adheres to the constraints. the RFC specifies that the constraints must be honoured for all subsequent certificates in the chain.


I would argue with separation of concerns then :smile: (reading from the frame with the slot versus getting the frame). But yeah in this case it's fine, I would have a stronger objection if a lot of code got in there.


Thank you for the additional improvements!

Should the line that is added have an ulimit value of 210 instead of 212?

+			u_limit = (sb.dsp.highspeed ? 233/*44.1KHz*/ : 210/*22.5KHz*/);
 		else
-			u_limit = (sb.dsp.highspeed ? 233/*44.1KHz*/ : 212 /*23KHz*/);
+			u_limit = (sb.dsp.highspeed ? 233/*44.1KHz*/ : 212/*22.5KHz*/); <---------


This check is already handle by DatabaseUtils.executeLargeInputs


Why do you need that ?


private?


Ja, god idé! tenker også at det er minimalt hva vi trenger eksponere utad. La inn depends_on på de containerne som avhenger av andre, mest fordi det gjør det lett å illustrere, men også siden det brukes av docker-compose til å sikre rekkefølgen av oppstart/omstart av ting.

Tredelingen av nett du nevner høres fornuftig ut, bare sett igang!


Не уверен, что этому классу нужно быть публичным. Думаю, он должен быть `private`.


while UseAlpha might not work with other ports, it would be a good idea to use the depth of the origin Bitmap for the new bitmap too. (so if the old one is 32, the new one should be 32 too)
also imo wouldn't it be better if the RescaleBitmap function would be part of wxBitmap too?


Well apart from tabs ,is there actually any code changed?


- renamed ContactAddress to DraftEmailer
- removed the null check from ctor


我先去掉这个选项


using the select by keys, you wouldn't have need to load active rules dto I think


Вот новая версия этой функции: https://github.com/ilyasProgrammer/pos-addons/blob/9.0/pos_multi_session/static/src/js/pos_multi_session.js


I get an error when I try to run your example with AR_90 angle.

I am using the example image below
![test image](http://i.imgur.com/6NkwQLc.png)

And it creates the dst image like below
![dst image](http://i.imgur.com/AIlONCc.png)

It tries to get the pixel value of dst image at [282,89] index for example, but the image has (284x1) pixels. I've changed the 89 to 0 in the code and rebuild the opencv and it worked like a charm. Is it possible that you've forgotten some module operation or something? Or am I making some mistake?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17384](https://teamcity.mesosphere.io/viewLog.html?buildId=146845&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:04:18


on_notification то есть


We should one day replace templateId by templateKey in the db !


а
            this.pos.db.save('bus_last', this.bus.last)
надо удалить или в другое место bus_last сохранять


Why ?


Commit by deepakpathania


据科学出巨款的


电话费觉得觉得可贷款就深V京东方开发贷款吧


Fixes #75


This saved me many hours of work,  thank you!


ok i see, then maybe simplify it as follows:

```
if ($deleteQuery) {
    if (not deletion confirmed and has data) {
        // abort
    } else (if deletion confirmed and backup failed) {
        // abort
   }

    // run query
}

```
I am only trying to nest less as possible :P


should I lock this dependency?


Why this change ?


removed


done


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17383](https://teamcity.mesosphere.io/viewLog.html?buildId=146844&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 238 Build time: 00:08:55


I'm not so sure, yet. It would make sense for consistency, but it depends on if the hardware caps non-highspeed DMA at 22050Hz or 23000Hz.


merged to add back support for full screen on OS X


Seems good all good +1:


extracted variable for validation result


Подправил


Thank you, however are you sure of this because even after erasing the cache it doesn't work as expected.

Blue arrow -> default table.
Red arrow -> csv table.

![tables](https://cloud.githubusercontent.com/assets/16919700/13348581/a659c246-dc74-11e5-9d8b-801e8242cfa5.jpg)


The context title is misleading here


removed preconditions. validation deferred to validate()


Oh, if we are going to use it here (which I think is fine) let's calle it `$halfBaseSize` then.


Formatting wrong


Yes we should revisit it. Damien?


A couple of notes:
1. it would be a temporary solution as it's a bit ugly and not convenient because the default vulcan auth middleware doesn't support multiple credential (user&password) pairs. This means we need to create a new front-end for the same routing rule for each _client_. A more refined solution would be implementing our own auth middleware.
2. We might not even need the UP _client_ credentials, depending on the status of the locking down our cluster for the host header based requests as well.


Mhhh you reverted it back to pass the repositry via the method instead via the constructor?


I can definitely hear further improvements with the latest changes. I can understand now why the sound cards had filtering. Next, I'm trying out the different sound card types with the filtering.


I would move the Globals.journalAbbreviationLoader = .... line to the corresponding test.


Nice find, I wonder if you could make a find-bugs/PMD rule for this one


Good day sir! I see you have made a correction, would you be willing to make a pull request and I will publish the fix :smile:


Why is this line needed?


I'll talk more about it in our meeting today.


Super thanks @klappradla!


De commit message is behoorlijk interessant ;)


Yep, it is officially BSD!
Cheers,
Remi-C

2016-02-25 17:33 GMT+01:00 Bruno Friedmann <notifications@github.com>:

> to help packaging ( as we maintainer have to indicate the spdx.org tag
> name) could we confirm it is
> http://spdx.org/licenses/BSD-2-Clause.html#licenseText
> ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pgpointcloud/pointcloud/commit/6e7dc8852084721cdb087c4e29da3f89a138ba2b#commitcomment-16331225>
> .
>


Could you please fix this ignore to only include the folders that are relevant for this project.


Did a PR on rpi-update to add a warning for this: https://github.com/Hexxeh/rpi-update/pull/201 but not merged yet.

Why are there two pages about kernel updates?
https://github.com/raspberrypi/documentation/blob/master/linux/kernel/updating.md

Maybe we should merge this into one. What do you think?


@rmeertens, ik denk dat @TNieuwdorp in de verkeerde repo zat te werken ;)


edit the README file


If Exception by decorated function alway will emit _UnexpectedSuccess.
would it be:
try
   func(*args, **kwargs)
except Exception:
   pass
else:
    raise _UnexpectedSuccess

the same in the following realExpectedFailure function


Ah, I didn't realize that. Then it's a necessary evil.


I would still require the Objects.requireNonNull!
The problem is that you mock the Loader which then returns a null repository. Just use mock(JournalAbbreviationRepository.class) and pass it as an argument to the used methods.


also include - https://trello.com/c/xx5VTsTJ


:+1: :smile:


Sssh, deze commits zijn er nooit geweest

-------- Original message --------
From: Wouter Bulten <notifications@github.com>
Date:26/02/2016  10:43  (GMT+01:00)
To: wouterbulten/acais-website <acais-website@noreply.github.com>
Cc: Thijs Nieuwdorp <thijsnieuwdorp@hotmail.com>
Subject: Re: [acais-website] Toevoegen tijdelijke pagina alumni-dag (3bcd2c1)

@rmeertens, ik denk dat @TNieuwdorp in de verkeerde repo zat te werken ;)

---
Reply to this email directly or view it on GitHub:
https://github.com/wouterbulten/acais-website/commit/3bcd2c1742e69cbb9e4ab661e8d4c8dfe06f6d26#commitcomment-16348149


Upload last successful WebView APK to Android File Host


You forget to reference to the issue you are closing


Yeah. Prolly best just to close this one ?


This make the test useless.
https://developer.gnome.org/glib/stable/glib-Testing.html#g-test-add-func


@svinota, this code is wrong because I can't figure out the memory structure in the kernel.
fq_codel seems to be a special snowflake because I've noticed it spawn 'phantom classes' during operation, which I assume is the cause for this: http://git.kernel.org/cgit/linux/kernel/git/shemminger/iproute2.git/tree/include/linux/pkt_sched.h#n755

It dumps a struct containing a union:
```
struct tc_fq_codel_xstats {
	__u32	type;
	union {
		struct tc_fq_codel_qd_stats qdisc_stats;
		struct tc_fq_codel_cl_stats class_stats;
	};
};
```

(why would it use xstats as a base, isn't that deprecated?)
We need a way of interpreting `type`, because `class_stats` and `qdisc_stats` contain different types of values, the structure of which has already been outlined in `fq_codel_qd()` and `fq_codel_cl()` above.

If you run this commit, you'll see it return two instances of either structure with different values, and they don't correspond to what's displayed in `tc qdisc -s`. Any help would be appreciated!


It creates a new `ContentStorePayloadVersion` for that `ContentItem` so that one exists in the presenter. There is a factory now that I've used elsewhere which would probably be more obvious.


Can we use lodash isFunction here?


ERROR: 'listener' hides a field.


ERROR: 'optionsViewId' hides a field.


ERROR: Missing a Javadoc comment.


ERROR: [STYLE][FE_FLOATING_POINT_EQUALITY] Test for floating point equality in org.fossasia.openevent.utils.RecyclerItemInteractListener.onInterceptTouchEvent(RecyclerView, MotionEvent)
http://findbugs.sourceforge.net/bugDescriptions.html#FE_FLOATING_POINT_EQUALITY


WARNING: [ContentDescription] [Accessibility] Missing `contentDescription` attribute on image
 Non-textual widgets like ImageViews and ImageButtons should use the `contentDescription` attribute to specify a textual description of the widget such that screen readers and other accessibility tools can adequately describe the user interface.

Note that elements in application screens that are purely decorative and do not provide any content or enable a user action should not have accessibility content descriptions. In this case, just suppress the lint warning with a tools:ignore="ContentDescription" attribute.

Note that for text fields, you should not set both the `hint` and the `contentDescription` attributes since the hint will never be shown. Just set the `hint`. See http://developer.android.com/guide/topics/ui/accessibility/checklist.html#special-cases.


WARNING: [RtlHardcoded] Consider adding `android:layout_marginStart="-50dp"` to better support right-to-left layouts
 Using `Gravity#LEFT` and `Gravity#RIGHT` can lead to problems when a layout is rendered in locales where text flows from right to left. Use `Gravity#START` and `Gravity#END` instead. Similarly, in XML `gravity` and `layout_gravity` attributes, use `start` rather than `left`.

For XML attributes such as paddingLeft and `layout_marginLeft`, use `paddingStart` and `layout_marginStart`. *NOTE*: If your `minSdkVersion` is less than 17, you should add *both* the older left/right attributes *as well as* the new start/right attributes. On older platforms, where RTL is not supported and the start/right attributes are unknown and therefore ignored, you need the older left/right attributes. There is a separate lint check which catches that type of error.

(Note: For `Gravity#LEFT` and `Gravity#START`, you can use these constants even when targeting older platforms, because the `start` bitmask is a superset of the `left` bitmask. Therefore, you can use `gravity="start"` rather than `gravity="left|start"`.)


Indeed.


Подправил


I do not understand: PulseAudio **was** optional before, but since this commit I cannot any longer compile on my Linux Mint 17.3 from which I have removed the damn PulseAudio!


The parent is defined when indexing at line 120. It does not need to be part of _source.


So, in this case, I'd better use `one_or_none()`.
Thanks.


i can't think of anything but will let @dannyroberts weigh in


ResourcePropertySource can retrieve file from given location (classpath, file system or even http I think). I will add the logging.


Are we sure this is not needed any more? It is been a long time, but if I remember correctly the syslog forwarder needs to know the port to send the logs to.


ERROR: 'cast' is not followed by whitespace.


ERROR: 'sessions' hides a field.


ERROR: 'tabPos' hides a field.


ERROR: Name 'SEARCH' must match pattern '^[a-z]([_a-zA-Z0-9]+)*$'.


ERROR: Unused import - org.fossasia.openevent.utils.RecyclerItemClickListener.


ERROR: 'listener' hides a field.


ERROR: 'optionsViewId' hides a field.


ERROR: Missing a Javadoc comment.


WARNING: [UnusedImports] Avoid unused imports such as 'org.fossasia.openevent.utils.RecyclerItemClickListener'
http://pmd.sourceforge.net/pmd-5.2.3/pmd-java/rules/java/imports.html#UnusedImports


ERROR: [PMD-CPD] 65 lines duplicated.
app/src/main/java/org/fossasia/openevent/activities/SessionDetailActivity.java


ERROR: [PMD-CPD] 15 lines duplicated.
app/src/main/java/org/fossasia/openevent/activities/SessionDetailActivity.java


WARNING: [SimpleDateFormat] To get local formatting use `getDateInstance()`, `getDateTimeInstance()`, or `getTimeInstance()`, or use `new SimpleDateFormat(String template, Locale locale)` with for example `Locale.US` for ASCII dates.
 Almost all callers should use `getDateInstance()`, `getDateTimeInstance()`, or `getTimeInstance()` to get a ready-made instance of SimpleDateFormat suitable for the user's locale. The main reason you'd create an instance this class directly is because you need to format/parse a specific machine-readable format, in which case you almost certainly want to explicitly ask for US to ensure that you get ASCII digits (rather than, say, Arabic digits).

Therefore, you should either use the form of the SimpleDateFormat constructor where you pass in an explicit locale, such as Locale.US, or use one of the get instance methods, or suppress this error if really know what you are doing.


WARNING: [UnusedAttribute] Attribute `fontFamily` is only used in API level 16 and higher (current min is 15)
 This check finds attributes set in XML files that were introduced in a version newer than the oldest version targeted by your application (with the `minSdkVersion` attribute).

This is not an error; the application will simply ignore the attribute. However, if the attribute is important to the appearance of functionality of your application, you should consider finding an alternative way to achieve the same result with only available attributes, and then you can optionally create a copy of the layout in a layout-vNN folder which will be used on API NN or higher where you can take advantage of the newer attribute.

Note: This check does not only apply to attributes. For example, some tags can be unused too, such as the new `<tag>` element in layouts introduced in API 21.


FATAL: [MissingTranslation] "`menu_schedule`" is not translated in "hi" (Hindi), "ro" (Romanian), "zh" (Chinese), "zh-CN" (Chinese: China)
 If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.

If the string should *not* be translated, you can add the attribute `translatable="false"` on the `<string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore="MissingTranslation"` attribute.

By default this detector allows regions of a language to just provide a subset of the strings and fall back to the standard language strings. You can require all regions to provide a full translation by setting the environment variable `ANDROID_LINT_COMPLETE_REGIONS`.

You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale="languageCode"` for the root `<resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)


FATAL: [MissingTranslation] "`filter_sessions`" is not translated in "hi" (Hindi), "ro" (Romanian), "zh" (Chinese), "zh-CN" (Chinese: China)
 If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.

If the string should *not* be translated, you can add the attribute `translatable="false"` on the `<string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore="MissingTranslation"` attribute.

By default this detector allows regions of a language to just provide a subset of the strings and fall back to the standard language strings. You can require all regions to provide a full translation by setting the environment variable `ANDROID_LINT_COMPLETE_REGIONS`.

You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale="languageCode"` for the root `<resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)


FATAL: [MissingTranslation] "`select_all`" is not translated in "hi" (Hindi), "ro" (Romanian), "zh" (Chinese), "zh-CN" (Chinese: China)
 If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.

If the string should *not* be translated, you can add the attribute `translatable="false"` on the `<string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore="MissingTranslation"` attribute.

By default this detector allows regions of a language to just provide a subset of the strings and fall back to the standard language strings. You can require all regions to provide a full translation by setting the environment variable `ANDROID_LINT_COMPLETE_REGIONS`.

You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale="languageCode"` for the root `<resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)


FATAL: [MissingTranslation] "`select_none`" is not translated in "hi" (Hindi), "ro" (Romanian), "zh" (Chinese), "zh-CN" (Chinese: China)
 If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.

If the string should *not* be translated, you can add the attribute `translatable="false"` on the `<string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore="MissingTranslation"` attribute.

By default this detector allows regions of a language to just provide a subset of the strings and fall back to the standard language strings. You can require all regions to provide a full translation by setting the environment variable `ANDROID_LINT_COMPLETE_REGIONS`.

You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale="languageCode"` for the root `<resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)


FATAL: [MissingTranslation] "`add_to_calendar`" is not translated in "hi" (Hindi), "ro" (Romanian), "zh" (Chinese), "zh-CN" (Chinese: China)
 If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.

If the string should *not* be translated, you can add the attribute `translatable="false"` on the `<string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore="MissingTranslation"` attribute.

By default this detector allows regions of a language to just provide a subset of the strings and fall back to the standard language strings. You can require all regions to provide a full translation by setting the environment variable `ANDROID_LINT_COMPLETE_REGIONS`.

You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale="languageCode"` for the root `<resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)


WARNING: [LogNotTimber] Using 'Log' instead of 'Timber'
 Since Timber is included in the project, it is likely that calls to Log should instead be going to Timber.


WARNING: [Overdraw] Possible overdraw: Root element paints background `@color/more_background` with a theme that also paints a background (inferred theme is `@style/AppTheme`)
 If you set a background drawable on a root view, then you should use a custom theme where the theme background is null. Otherwise, the theme background will be painted first, only to have your custom background completely cover it; this is called "overdraw".

NOTE: This detector relies on figuring out which layouts are associated with which activities based on scanning the Java code, and it's currently doing that using an inexact pattern matching algorithm. Therefore, it can incorrectly conclude which activity the layout is associated with and then wrongly complain that a background-theme is hidden.

If you want your custom background on multiple pages, then you should consider making a custom theme with your custom background and just using that theme instead of a root element background.

Of course it's possible that your custom drawable is translucent and you want it to be mixed with the background. However, you will get better performance if you pre-mix the background with your drawable and use that resulting image or color as a custom theme background instead.


WARNING: [Overdraw] Possible overdraw: Root element paints background `@android:color/white` with a theme that also paints a background (inferred theme is `@style/AppTheme`)
 If you set a background drawable on a root view, then you should use a custom theme where the theme background is null. Otherwise, the theme background will be painted first, only to have your custom background completely cover it; this is called "overdraw".

NOTE: This detector relies on figuring out which layouts are associated with which activities based on scanning the Java code, and it's currently doing that using an inexact pattern matching algorithm. Therefore, it can incorrectly conclude which activity the layout is associated with and then wrongly complain that a background-theme is hidden.

If you want your custom background on multiple pages, then you should consider making a custom theme with your custom background and just using that theme instead of a root element background.

Of course it's possible that your custom drawable is translucent and you want it to be mixed with the background. However, you will get better performance if you pre-mix the background with your drawable and use that resulting image or color as a custom theme background instead.


WARNING: [Overdraw] Possible overdraw: Root element paints background `@color/title_picker_bg` with a theme that also paints a background (inferred theme is `@style/AppTheme`)
 If you set a background drawable on a root view, then you should use a custom theme where the theme background is null. Otherwise, the theme background will be painted first, only to have your custom background completely cover it; this is called "overdraw".

NOTE: This detector relies on figuring out which layouts are associated with which activities based on scanning the Java code, and it's currently doing that using an inexact pattern matching algorithm. Therefore, it can incorrectly conclude which activity the layout is associated with and then wrongly complain that a background-theme is hidden.

If you want your custom background on multiple pages, then you should consider making a custom theme with your custom background and just using that theme instead of a root element background.

Of course it's possible that your custom drawable is translucent and you want it to be mixed with the background. However, you will get better performance if you pre-mix the background with your drawable and use that resulting image or color as a custom theme background instead.


WARNING: [UnusedResources] The resource `R.color.item_session_divider` appears to be unused
 Unused resources make applications larger and slow down builds.


WARNING: [AlwaysShowAction] Prefer "`ifRoom`" instead of "`always`"
 Using `showAsAction="always"` in menu XML, or `MenuItem.SHOW_AS_ACTION_ALWAYS` in Java code is usually a deviation from the user interface style guide.Use `ifRoom` or the corresponding `MenuItem.SHOW_AS_ACTION_IF_ROOM` instead.

If `always` is used sparingly there are usually no problems and behavior is roughly equivalent to `ifRoom` but with preference over other `ifRoom` items. Using it more than twice in the same menu is a bad idea.

This check looks for menu XML files that contain more than two `always` actions, or some `always` actions and no `ifRoom` actions. In Java code, it looks for projects that contain references to `MenuItem.SHOW_AS_ACTION_ALWAYS` and no references to `MenuItem.SHOW_AS_ACTION_IF_ROOM`.


The very fact that masked arrays were even accepted was incidental and it was just ignoring the mask. A pull request with the wording describing the impact of this change in the "backwards incompatible changes" section of the 0.17.0 release notes can be gratefully accepted.


gute Idee :+1:


哥们封装的不错


我觉得不用导航控制器应该也可以用这个,再考虑下没有导航控制器的时候


asdf


yes, only meet such kind of issue in listed guest. No meet in Win7 guest.


We should have some kind of clarification or enumeration here which fetches are covered and which are not. E.g., I think it might be better to have the note below be here, since otherwise you'll have to read through everything to figure out the architecture.


Apparently changing permissions result in so many changes


надо в конструкторе всегда иницализировать списки. BackupFiles  = new List<FileInfo>();


#1 fixed


Could remove a lot of duplication:
```python
if not settings:
    settings = OpenClinicaSettings()
    settings.domain = domain.name
settings.study.is_ws_enabled = self.cleaned_data['is_ws_enabled']
etc....
```


Надо сделать его protected virtual


OK, i miss the splitToList. Fixed.


лучше переименовать это свойство в GetBackupDirectoryPath


http://wiki.garrysmod.com/page/Global/SysTime

> Returns a highly accurate time since the start up, ideal for benchmarking.

http://wiki.garrysmod.com/page/Global/RealTime
> You should use this function (or SysTime) for timing real-world events such as user interaction, but not for timing game events such as animations.

Where the fuck are you getting the idea that frametime or realtime would be better? Frametime returns the time it took to render the **last** frame. It would be as useful for measuring the performance of the above loop as `player.GetAll()[1]:Deaths()`.


Thank you. The idiom wasn't clear to me.  Notice how useful this is when we want to pass on an optionally omitted parameter to our client Request types.


Лучше как и везде это сделать методом, а не свойством


Ohh yeah. I made a small mistake in indexing. Indices for columns should not be `theta - 1` . It should be indexed based on the mode. Thanks for pointing it out.

I was thinking about getting radon transform for all angles between 1 and 180. If you try to use ` mode = AR_1_45 | AR_46_89 | AR_90 | AR_91_135 | AR_136_179 | AR_180 `, it will work properly and it computes transform for all angles between 1 and 180.

I will fix this bug and push it soon. Thanks again.


I looked up the apply method for Scala and found this StackOverflow answer http://stackoverflow.com/questions/9737352/what-is-the-apply-function-in-scala. Apparently all functions inherently have an apply method and anything with an apply method can be used as a function. Therefore if you give an object an apply method, you can use it as a function.


This variable isn't necessary anymore. Just "throw new AuthenticationException(..."


I agree, I don't like the angle brackets around the + and *


This commit (specifically moving this line out of the OpenDisplay call, thus making it impossible for m_adapter to ever be anything but `nullptr`) breaks multi-GPU support. If you want this refactored, then I suggest into smaller classes that we can compose, and/or small functions.


[![Three Women](http://api.artsmia.org/images/2299/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=2299)


No deep copies are required, no :) yet. I just wanted a version that
allowed me to remove the lodash dependency, keep browser compatibility. I
don't think lodash's clone is deep by default either?
On 26 Feb 2016 23:57, "Phil Plückthun" <notifications@github.com> wrote:

> Found another thing ^^ This is obviously close to an assign method's
> functionality. I'd actually expect a clone method to deeply clone an
> object, though, instead of this shallow copy.
>
> Could be replaced by just using a custom assign helper, I guess. I'm not
> aware of any parts of faux-dom needing a deep copy. Or are there some?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Olical/react-faux-dom/commit/4873631adae7ccf7dc3353d72964335e4e7ecfb8#commitcomment-16366832>
> .
>


Why not put this part in the base Dockerfile, too?


Why is this a premature optimization and `j = methodsOrder.length` wasn't?


In the future, it'd be better to run pronto after each commit and squash style changes into the commits that introduced the bad style. But this is fine for now


it looks interesting for me:
separate thread, std::chrono std::atomic, TEH std::mutex, std::cout

But let me suggest next links:
https://github.com/CppCon/CppCon2014/tree/master/Presentations/Where%20did%20my%20performance%20go
https://www.youtube.com/watch?v=tOxi-cH7n8A
it is about mmap file and small (16-bytes) timestamps for events.
Also we have:
https://github.com/epam/nfstrace/blob/master/src/utils/profiler.h


why not `char temp[sizeof("999.999.999.999")];` then it is obvious why its this size.


Also, you never want to just do an "abort" you can throw an exception... but there needs to be some error.


That also was a premature optimization and shouldn't be done.


The functionality of Like/Unlike and Follow/Unfollow are now broken.  Please write that in the issue next time you know there are issues. And please fix those.


@duke12468 I no longer use Skype, you can join the discord room here,
https://discordapp.com/channels/152905170009718785/152905170009718785


rubbish


Why would that be?


Right, makes sense. I can just revert previous text for type -- I believe it covers the behavior we want? Also, I'll add separate points for `as`, just as the new text now has.. Yay?


I can share a code inspired by the presentation but it is under NDA and it is Win32 only.


Easiest thing to do if you want to force a key like this... is

```
param name="arguments.data.collectionConfig" required="true";
```

I think you can even make sure it is the right type:
```
param name="arguments.data.collectionConfig" required="true" type="json";
```

Instead of doing the structKeyExists


Went through the same "syncing exercise" as yesterday:  deleted origin/dev branch, and created a new "dev" branch off upstream dev (acthp/ucsc-xena-client/dev).  This time, "hooked-up" local origin/dev to remote origin/dev, and then force-pushed to remote dev branch.


Replacing `x-bind` with `bind` is not recommended. `x-` is a prefix indicating that its a user-space attribute and not a browser native attribute, to prevent UA's from having any specific treatment of those values.


isRegExp implies that it is a boolean which `test` will return.


1) In terminal: rails generate migration AddStateToTasks state:string
2) In ./app/config/db/.._add_state_to_tasks, added default: 'to_do' string
3) rake db:migrate

In Rails Console:
4) t = Task.new
5) t.content = "This is the content"
6) Task.last
7) when you type t.user, it is nil right?, so associate t.user = User.first
8) t.save

Sublime:
9) In ./app/views/tasks/_form.html.erb, displayed the new 'state' field
10) In ./app/views/tasks/show.html.erb, added @task.state erb
11) In ./app/views/tasks/index.html.erb, added State header and task.state erb into tbody
12) In ./app/controllers/tasks_controller.rb, added ... permit(:content, :state)
13) In ./app/controllers/tasks_controller.rb, in def index I added @to_do, @doing, @done
14) In ./app/views/tasks/index.html.erb, added 3 tables and changed title and tbody to @to_do, @doing, @done


Async events don't really matter though because they don't block the UI and plugins can do literally anything there.

This would be the equivalent of a plugin hosting a thread and just doing everything there when an event occurs instead.


Keith and I were thinking that it might make sense to make this queue unbounded. The BW will already be limiting the memory used (my "10" was just pseudo-code; a guess number).


A wait, no, got it backwards: http://stackoverflow.com/questions/17154830/what-are-the-advantages-of-using-data-rather-than-x-prefix-for-custom-attribut

Ignore me.


plau


It's not a bug. Messages are already linked with usernames on the server. As it stores `Maybe String` for each connection.


Appearing together is handled by the indices, not the order in which the conditions are evaluated. I'll add a test.


@viniciusjarina Thank you! I'll fix it.


helpID should be in the action.payload, not hanging off action.  action should only have 2 properties: type and payload.


Ah, good point. I'll change it.


That looks like the right binary. I don't think that config should would work that binary:

http://nginx.org/en/docs/http/ngx_http_v2_module.html

Have you done a full stop and start of nginx?


Проверил на разных языках - везде ок отображается. Точно арабский был проблемный?
> On Feb 26, 2016, at 7:56 PM, Alexander Marchuk <notifications@github.com> wrote:
>
> Ну, у арабов же свои крючки для цифр. Мы натыкались на эти грабли.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/yunikkk/omim/commit/7914631aeada19ef39528d05e322b3225a94821f#commitcomment-16357354>.
>


I know it's nitpicking, but will you please name it helpId?


You can just do this right underneath bool pv=true

bool pv = true;
Reductions[!pv][imp][d][mc] = 0 * ONE_PLY;
Reductions[pv][imp][d][mc] = 0 * ONE_PLY;


I prefer { on same line but I've worked on projects with both. Tabs are better IMO, spaces get annoying. Eclipse doesn't seem to have many import configs. I don't think I can have the auto import without everything else the import organization offers.


Видимость контролируется всё равно уже во фрагменте


This is pretty great, and probably closes @YanaW's pull request #8, right?


The indenting is wrong.  Also what you are doing here is to extend the Espressif validation for hostnames so I would just _KISS_ and  do that AND explain your validation rule.  Also don't bother to double up on  checks alread in the SDK wifi routine.
```C
static bool wifi_sta_sethostname(const char *hostname, size_t len)
{
  int i;

  if (!isaplhpa(hostname[0]) || hostname[len-1]=='-' || len > 32)
    return false;

  for (i=1; i<len-1; i++)
  {
    if (!isalnum(hostname[i]) || hostname[i]=='-'))
      return false;
  }
  return wifi_station_set_hostname((char*)hostname);
```


link isnt working for some reason @neetVeritas


this should be:
`size: PropTypes.oneOf(['small', 'large'])`

this allow for additional sizes if needed.


It's not a major issue tbh, if we have some more problems with it I will actually do something about it


@ericchiang is going help we with this once we have the smaller issues on this commit addressed.


Thanks!


Shoot, I didn't notice this was already merged. Feel free to PR https://github.com/turadg/react-codemod/tree/simplify-sort-comp


you have to click invite members and the libnk has to be liek this https://discord.gg/0qxcoZ8rpta5DhLX
@neetVeritas


Added feature proposals


The Lua way is to throw errors that are unexpected rather than leaving it to app programmer to litter the app code for checking for unusual returns where they will either throw an erro or not bother to check.  so I personally  wouldn't bother returning a false status. If the programmer wants to catch this, then its easy to do a pcall.  There's no point in this debug, since the throw tells you the same.  BUT update the docn if you accept this.
```C
static int wifi_sta_sethostname_lua( lua_State* L )
{
  size_t len;
  const char *hostname = luaL_checklstring(L, 1, &len);
  luaL_argcheck(L, wifi_sta_sethostname(hostname, len), 1, "Invalid hostname");
}


Yeah. I'm afraid one day I'll accidently click my auto formatter hot key and screw up loads of junk.


OK, I'll send the test in few minutes.


I see the purpose is to delete the plucker before processing alarm-related stuff just as you would in the case of a timeout. So I think that this makes sense, but it's still failing on timing-related tests.


Not that it matters since I changed it anyway, but yes. The ones we care about do handle default macro values as MSVC. CLang follows what GCC (it is written in the CLang documentation). And here is how GCC expands the macro defines:
From http://gcc.gnu.org/onlinedocs/gcc/Preprocessor-Options.html
-D name
Predefine name as a macro, with definition 1.


well done.
Thêm vị trí của cú click + vị trí hiện tại của cái xe lăn --> đường đi của nó.


If I add another PROVIDER will this also need to be updated?


Oh right. it's not even valid json input.


xvxv


I don't recall exactly, but when I tried loopargs.length it broke or something.


After this change, I'm now having my fingers match other fingers. For example, my right index finger can now match all 10 different enrolled fingers randomly. The images are too compacted now.


The ClassId enum is automatically updated from the providers macro, which should cause a warning here that not call cases in the enum are handled


Why are you adding these webviews here?


Stand Library from C?


what's self?


GBA used an ARMCPU?


GBA ran at 60fps????


Nice Work @ariel-  and @Treeston  :+1:


you're self in it???


Note that dotty deals with that correctly :) https://github.com/lampepfl/dotty/blob/0eecb7189188e6ce379b8840749abb1e0241035b/src/dotty/tools/dotc/transform/Erasure.scala#L188-L197


fix-window-frame-calcs-5095 (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1570 of 1571 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.94%25-yellow.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.8%25-green.svg)


You might want to set up your editor to use tabs, not spaces :]


k


Should lsusb -vv of your device


This is denormed because Mongo and DDP


For crying out loud how did that happen!
I've undone the last change!


I get that async events don't block the UI, and profiling them is less important. It would still be convenient to know how long a plugin spends responding to certain events, particularly something like `on_query_completions()`. Obviously plugins can time it for themselves, too.


Message and Follow/Unfollow buttons are broken. Were you checking it when you made changes?  Please report unfinished changes in the issue section next time.   And please fix these.


BSD or MIT would be really nice. :)


Part of #25.


Close #25.


And also removed some unnecessary'using' statements.

Bootstrapper is needed to use EventAggregator.


Pozor, HTMl do views rozhdně nepatří - hoď si to do šablony.


@jerryz123: Here's the commit I promised you on Wednesday -- sorry I got distracted and forgot about it until now :-)


Minor:  A one-liner alternative to the if/else block

    return ($row != null && password_verify($password, $row['password_hash']));

This is a good opportunity to make you aware of the concept of "short circuit evaluation":
https://en.wikipedia.org/wiki/Short-circuit_evaluation

Basically:
- In an AND condition, if the first condition is false, the second is not evaluated.

    'b' not evaluated if 'a' is false:  if (a AND b)
- In an OR condition, if the first condition is true, the second is not evaluated:

    'b' is not evaluated if 'a' is true: if (a OR b)


@KimTranjan Você pode implementar isso no Branch DNX também? pois o [fork que você pegou](https://github.com/BoletoNet/boletonet/pull/144) vai dificultar o merge dessas modificações no DNX


What is the motivation to remove this dependency?  Would you object to something like a clean `Modsulator::Utils` gem that would remain the single place for this code to live, consumed by both modulator and dor-services?

I'm happy to make that happen, but disinclined to include this copied code in our current deploy effort.


Yes! Great work on this section, @sagudo and Alex.


oops


/* ----------------------------------------------------------------------------
 * Copyright © 2016 Jacob Yablonski <jyablonski@csu.fullerton.edu>
 * Released under the [MIT License] (http://opensource.org/licenses/MIT)
 * ------------------------------------------------------------------------- */

 /**
  * Tying everything together.
  *
  * Notes:
  * - We also include our test functions here, since there aren't that many.
  */
#include "Buffer.h"
#include "constants.h"
#include "neighborhood.h"
#include "shape.h"

  // ----------------------------------------------------------------------------
  // test functions

  /**
   * Test the `Buffer` object by creating one, drawing a few things on it, and
   * then rendering it to the screen.
   */
//   void test_buffer() {
//	Buffer b(TERM_SIZE_X, TERM_SIZE_Y);
//
//			// draw ellipse
//		for (unsigned int y = 0; y < b.size_y; y++)
//		 for (unsigned int x = 0; x < b.size_x; x++)
//		 if ((x - 40)*(x - 40) + (y - 10)*(y - 10) == 50)
//		 b.set(x, y, '*');
//
//			// draw border
//		for (unsigned int y = 0; y < b.size_y; y++) {
//		b.set(0, y, '|');
//		b.set(b.size_x - 1, y, '|');
//
//	}
//	for (unsigned int x = 0; x < b.size_x; x++) {
//		b.set(x, 0, '-');
//		b.set(x, b.size_y - 1, '-');
//
//	}
//
//			// render to screen
//		b.draw();
//
//}
//
/////**
////* Test the `Shape` object by creating one of each type (besides "empty"), and
////* tiling them on a buffer.
////*/
//void test_shape() {
//	Buffer b(TERM_SIZE_X, TERM_SIZE_Y);
//
//		Shape triangle("triangle");
//	Shape square("square");
//
//			 //draw shapes
//		for (int y = 0; y <= b.size_y - Shape::size_y; y += Shape::size_y) {
//		for (int x = 0; x <= b.size_x - Shape::size_x * 2; x += Shape::size_x * 2) {
//			triangle.drawToBuffer(b, x + Shape::size_x*((y + 0) % 2), y);
//			square.drawToBuffer(b, x + Shape::size_x*((y + 1) % 2), y);
//
//		}
//
//	}
//
//			// render to screen
//		b.draw();
//
//}

// ----------------------------------------------------------------------------

/**
 * Notes:
 * - Comment out the parts you aren't currently using.
 */
 int main() {
	//test_buffer();

			/* ------------------------------------------------------------------------*/

		//test_shape();

			 //------------------------------------------------------------------------

			// animate  neighborhood
	unsigned int size_x = TERM_SIZE_X / Shape::size_x;
	unsigned int size_y = TERM_SIZE_Y / Shape::size_y;
	Neighborhood(size_x, size_y).animate(1000);

			// ------------------------------------------------------------------------
	return 0;  // success

}


prosímtě používej EN názvy v proměnných tak, jak to je všude jinde - není větší binec, než česko-anglický slovník namísto zdrojového kódu.


Chtěl jsem si to zkusit u sebe, ale čistý checkout té větve nefunguje - chybí tam přidat forms.py a co jsem es díval do toho commitu, tak i nějaká změna v  urls.py


This function should return true at the end to match the false in the error case.


fix-window-frame-calcs-5095 (nealkruis) - Win64-Windows-7-VisualStudio-12: OK (2125 of 2127 tests passed, 1 test warnings)

 * 1 test had: ESO big diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO small diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.91%25-yellow.svg)


Tais brisando nos código ein amigo.


Reverted


Right I reverted the whole file to your latest version and it worked fine.


For me this doesn't work, i'm using:
      var file = path.join(base,src)
Anyone can confim?


Reverted,


Need a space before the handler block.


Looks good


> On Feb 25, 2016, at 9:23 AM, Jack Koenig <notifications@github.com> wrote:
>
>> Should we use the name Chisel3? We do have the version number to distinguish from Chisel2.
>>
>
> Although we have the version number as a way to distinguish specific Chisel versions, I don't think we should distribute Chisel3 as "Chisel" until we're ready for it to replace Chisel2 as the default version.
>
> I'll put together a suggestion for a migration plan that we can tear apart at the next Chisel meeting.
>

Here's a link people at Berkeley should be able to use to view the document:

	https://docs.google.com/a/berkeley.edu/document/d/1wDci3G5tM5BmqkE588VpTWiYcIUyQQbzfcqUUH-6o0A/edit?usp=sharing


Tutorial how to install OpenCV 3.1 in VS15:
http://muchomanual.com/Instalar-OpenCV-3-1-y-Visual-Studio-2015-/


Yep, will close #8. Thanks Parker and Yana!


commit try


You are specifying the first argument beeing `Enum.t`, but any pattern matching that will happen on it later is matching it as a list. So a `Set` or a `Map` are valid input by the spec, but will fail the pattern matching.

Therefore I'd suggest to change the spec to `@spec binary_search([a], a) :: non_neg_integer | nil when a: var` (or was it `where` in elixir?[^1])

[^1]: I do always confuse these two keywords between Erlang and Elixir…


Even if you didn't add documentation to the rest of the codebase yet, you should at least add the function documentation header for the new functions you add. Otherwise you're just accumulating more work for the future.


Nevynalézej kolo - zpracování dat z formulářů je docela komplexní úkol, pokud nejde o něco hodně specifického, už na to jsou napsané třídy. Pěkně si ve forms.py nadefinuj formulář a pomocí kwargů mu nastav atributy, jako je třeba required=True. Taky se tím zbavíš toho hlubokého zanořování.


Hey Jay -- I'm confused why you reverted this guy?


@stefanheimes feel free to merge


This should be called after `dmu_tx_assign()`.  Prior to this the tx isn't part of any txg.


Its not an array, be careful with that name, it does suggest some runtime-characteristics which a list can't hold (especially random access in O(1)).


If this truely is a hot-path, you'd be better off if you could find a way to reduce some of those to postfix-if instead. Will eliminate an extra scope which might have a performance effect ( make sure to test this hypothesis well :) )


Opinion:  'update' is more common in a codebase instead of the word 'change'.


FWIW, I find it extremely annoying when a class's fields are declared in the middle of the class, especially following method implementations. I always look for them just before the constructor and it's always the first thing I look at.


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#YeMEoEhYQxG4EODNtvY-YQ


eh i agree with this change. for now. as per workshop


2097-EP-8-5 (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


this would allow pasting unix timestamps into the form field, right?


Yeah, you could, with next instead of if() { } layers.


Obrigado. Vou atualizar


I will remove these.


2097-EP-8-5 (kbenne) - Win64-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


:+1:  Fair enough!


Iterates over the first half of the list.


hmm?


version_modified (macumber) - i386-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


This is an interesting one.  Try going into the `modules` folder. touch this file and do a make.  Now swap out everything to do with `-o` and swap `-c` for `-S`.  This gives you the source listing (trash this `.s` file when done or do an `xtensa-lx106-elf-objdump -d` on the object file.  You will see that this routine only generates 8 instructions inline vs. 5 outline and saving of 7 off 8 bytes.  This routine generates more than this, so in fact this would be better a macro or a static inline in a header.   OK, if we did this globally then it might be worth it, but the truth of space saving optimisations is often counter intuitive.


(esc_hardware[esc_index]->tim != esc_hardware[tx_index]->tim) never occurs for STM32F3Discovery


*`AssertionError`, and watch your mouth mister. :P


Hello Miguel

I'm attempting to migrate magento 1.9.x to magento 2.0.2 on a nginx web server.

To deal with this whole permissions issue I have written a script to parse out and change the permission settings based on the recommendations found in the various bug reports.

As I find other "obvious" errors I will report them here, to you, for inclusion in your fixes so that they are all in one place if that is acceptable to you.

With that said, you should probably also include: vendor/magento/module-deploy/Model/Filesystem.php
It contains the same error as:  app/code/Magento/Deploy/Model/Filesystem.php

[114]            self::PERMISSIONS_DIR,
[115]            self::PERMISSIONS_DIR

Regards
Glen


version_modified (macumber) - Win64-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


oh bollocks.


@sameera2004 please update XPCS-SAXS based on this latest  XPCS_SAXS_Feb_26.ipynb and Mask_Pipeline-Feb26.ipynb.


I cherry picked this commit.


I saw 180 reduction after implement "Kill hidden GDI pen created for each usercontrol instance". But how you get the count of "cold-start GDI count" 335? On my win7 (32bits), I run PhotoDemon.exe, I saw 1735 GDI Objects on build of 26/2/2016 and 1553 on build of 27/2/2016.


invalid according to what?


Does this commit message convey what the problem was?


ll - is longitude, latitude - think x, y (I fixed it in a separate commit)


Also for the `non_neg_integer | nil` we could define a type somewhere. It looks as if it will get a very common type. I'd suggest `Exads.position`.


computeFrontier(AllowingCFGChanges) sounds like it's performing the same operation but its return value has the opposite meaning. If an API is an imperative, then 'true' should always mean success. You could either wrap the result up in a struct, as I originally had it, with a "changedCFG" method, pass a separate flag, or simply rename this method to didComputeFrontierChangeCFG(Frontier&).


oh, i thought this was a mistake of mine in compatching a new CB, didn't see the context


Both files are the same, the path is different because you probably got the files from magento.com @glenara


Commit message is supposed to say '**all** context switching'.


Bone, not Bones


Thanks to https://github.com/anapsix/docker-alpine-java


Bone, not Bones (x4)


# the changes has no effect on the result


Certo @caduvieira. O @isaacvits provavelmente fez esse commit da máquina de desenvolvimento, onde estavam minhas credenciais. Revoguei as chaves no Github, apaguei da máquina de desenvolvimento e limpei o cache do git local da máquina. Um novo par de chaves deve ser criado para o ambiente de desenvolvimento.


__API Description file syntax error on line 2611__: base type 'Multiaddr' is not defined in the document (Autogenerated message from apiary.io)


Bone, not Bones

https://tos.neet.tv/items?q=bone+fragment it's a bag, so maybe it should be "Bone Fragments"


This doesn't even work. The MOTDPage is set through RunConsoleCommand but then you are fetching the ConVar _after_ this so you are not saving the old value.

The hook should be okay to remove completely now though.


Unbegrenzte -> Unbegrenzte**r**


I keep remembering there's some version-compatibility problem with old perls surrounding `goto`, but I keep forgetting exactly what it is.

Seeing we presently have smokers indicating `5.6` support, I don't want to make that worse. But @shadowcat_mst can probably be a better critique of the code.


@olavorn, este meu fork possui muitas modificações que eu preferi não enviar para o projeto original, pois contém algumas features/modificações que acabariam gerando breaking changes de algumas funcionalidades que já são utilizadas por usuários do Boleto.NET.
Essa alteração em específico também pode acabar gerando breaking changes para as pessoas que já utilizam a library, porém, como o DNX já vai criar alguns breaking changes, então acho justo eu criar um PR com essas alterações, já que elas são benéficas para quem gera arquivos de remessa.

Em relação ao merge que eu dei com o projeto original, ficarei mais atento nos merges, pois imaginei que os PR eram revisados.


@BigNoid Duplicate of VAR on [line 670](https://github.com/BigNoid/Aeon-Nox/blob/3441abe32b2777bc38f72c86738975a5b207fe90/1080i/variables.xml#L670) :wink:


Cheers :).

On Fri, Feb 26, 2016 at 10:12 PM, Steve973 <notifications@github.com> wrote:

> This should be replaced by the maven dependency plugin, which does not
> truncate the long path name. The pom for the remote karaf module does this.
> See my commit at Steve973@be0303d
> <https://github.com/Steve973/arquillian-container-osgi/commit/be0303d52df56ce74c027bc73a0e50987e649994>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wav/arquillian-container-osgi/commit/122b53c770c6a31aa2e588bd500f6346915f09b7#commitcomment-16350302>
> .
>


fix-window-frame-calcs-5095 (nealkruis) - i386-Windows-7-VisualStudio-12: OK (2125 of 2127 tests passed, 1 test warnings)

 * 1 test had: ESO big diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO small diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.91%25-yellow.svg)


nit: too much indentation


One other problem I can see is falling back to mcrypt.  If urandom is not readable, it is pointless to try to fallback to mcrypt with a urandom flag.  In fact, this hangs on my environment (PHP 5.6 + mod_mcrypt) when urandom is not readable due to open_basedir.


ok to change


Sounds good.


This does unnecessary work and allocations even when `endpoints` hasn't changed. If you extend the suite of traits in c.t.f.loadbalancer (Balancer and Updating) and implement a simple Distributor and Node, you get observation for free and a new distributor is created only when necessary.


You can just delete if the modified/created dates exist and then recreate those. That would be simpler logic. I think you also need to delete the published date if it exists in the new_resource which would be the case if a version is created from a published resource.


oops, will fix


Any reason why this is wrapped? :)


It's confusing here because these addresses are in ns, but `make_entries` divides them by 4 to reference 4ns blocks instead, so the offset is `JT_MIN_CLK_CYCLES_BETWEEN_OPCODES * 4` while the comparison code above checks again just `JT_MIN_CLK_CYCLES_BETWEEN_OPCODES`. I think it would help if in these tests we rename these addresses like `from_addr_ns`, which is the same naming convention used internally in `make_entries`. I'm ok if we do that change in a separate PR though.


it worked :dancer:  :D
thanks!


This logic is  wrong.  `luaL_ref()` adds the (reference to) the object into the the registry and returns an index for this.  It is always an index to an unused element so if `*cb_ref == ref` then something is _very_ wrong, so this test is redundant.

What might happen is that the previous function is the same as the current so if you _relly wanted to avoid a frequent unref/ref then is might be worth first doing a `lua_getref()` to see if the current function is the same as the last, but nowhere else in the app does anyone bother doing this.


creating a new branch


OK, I'll add a comment.

Short answer: compiling with `dotty/run` is not enough. You also need to link and then run with a JS interpreter. This is hard to do on the command-line without using the `ScalaJSPlugin` auto-plugin.


We would also converge to each node receiving 1/nth the traffic if we pick a random node on each request, right? It might be worth considering this since acquiring a shared lock here might represent a point of contention because all requests flow through here (we could use a thread local rng).


This commit and the previous one should probably be squashed together, editing the code entirely in one commit, and then doing a complete do-over in the next just serves for confusion.


Indeed, this is outdated. The initialization to zero is now part of the emitter, at link time.


can you fix the etcd port.


Array and Strings


You still need to put in the function for dues....As I did this differently, I'd like to know how we can add dues and tax rate without messing this up...All I essentially did was add more private data members to member.h and executive.h


This commit is fixed here: https://github.com/gsstudios/initramfs3/commit/cbfc951b1918493415b4352f990f73a445ba05e7


Building a pretty UI was never my strong point, but this attempts to integrate the install UI with the main form.

Looks much nicer than the first form though!


different randomiser not using the seed properly?


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#a2Zly0DNRSewXhr1KPEB-w


Good to know. I'll add a comment to revisit this for dotty when I get to enable this code. Here the logic would probably be the same, but [`fromAny`](https://github.com/scala-js/scala-js/blob/master/compiler/src/main/scala/org/scalajs/core/compiler/GenJSCode.scala#L3227-L3235) will probably have to be adapted.


Looks good.


I have mixed feelings about the utility of merges in a pull request. Typically I've always found it more useful to have linear histories where ever possible, and this history is very trivially linearizable.

Though I see call for it sometimes in repos ... its utility seems much less objective in a pull request.


Indeed.


I added a few lines for UVSQ, copying upon UPSud. Sorry for doing it last minute.


"was", "were" and "damage" are not translated :P


Yes, until Scala.js 0.6.8 is released and we can use that version.


This is great! :+1:


It is commented out. It is called by the (commented out) code 15 lines higher. I can also remove it entirely for now, if you prefer.


Does any Multi-Byte Unicode String search support yet?


Oh!
On the machine I'm using for this editing I downloaded magento 2.0.2 and unzipped it.
This where this file name is located.
On my magento server I used the composer and got the same result.

Where are your installation files coming from that would have different paths?
I'm confused?
Glen


@VoyagerOne: please start a test when the tuning finishes.


beh, un po li link verso l'esterno migliorano il ranking, forse da lasciare questo link...


`table_add_int()` and `table_add_string()` are definitely the wrong names.  These are external and part of the `wifi` namespace.  _All_ wifi externs should have the `wifi_` prefix.  These aren't adding to a table anyway, they are adding fields to the stack so this would better `wifi_add_int_field()' and the next `wifi_add_sprintf_field()` or something like that.


I haven't come around to looking into tail-rec methods yet. I'll remove the comment and the commented out code below for now, then, if you say the logic will change anyway.


yes


If you are going to keep the new title "Model Resources" for the model resources group, please include the Model Program resource to this group.


I think it did happen to me earlier, but now I'm not sure anymore. I will check again.


This can be written as: `return Carbon::instance($value);`
See also: https://github.com/briannesbitt/Carbon/blob/master/src/Carbon/Carbon.php#L253


Nope it is still unsupported
On Sat, 27 Feb 2016 at 02:22, cxj98 <notifications@github.com> wrote:

> Does any Multi-Byte Unicode String search support yet?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/x64dbg/x64dbg/commit/9ac27751d9f023adcb810e8610d085fe26706922#commitcomment-16367682>
> .
>


It would be great if you change "Model resources" to "Model Resources"


Yes, it is. What do you want me to do about it?


The scala language:
```scala
scala> class A { def foo(): Int = 1 }
defined class A

scala> class B extends A { override def foo(): Int = 2 }
defined class B

scala> class C extends B { override def foo(): Int = super[A].foo() }
<console>:9: error: A does not name a parent class of class C
       class C extends B { override def foo(): Int = super[A].foo() }
                                                     ^
```
I haven't checked whether this is allowed in dotty, though.


Because I was prototyping. It should fail at compile time, indeed.


LGTM.


@Olical Just looked it up and indeed `clone` is copying shallowly :smile: I don't work with lodash that often.

But still, I guess it's unnecessary if there's `assign({}, x)`


I don't see how this can be constexpr.


teste


If I specify long load balancer DNS names, then it is throwing this error:

ProxyPass worker hostname (internal-spinnakertest-rosco-1698591640.us-west-2.elb.amazonaws.com) too long

It seems that there is a bug in apache2 related to this: https://bz.apache.org/bugzilla/show_bug.cgi?id=53218


It is supported, on the contrary. `"1".toString()` is emitted as a normal method call in the IR, and is therefore an `Apply(StringLiteral("1"), Ident("toString__T"), Nil)`. So it actually has *nothing* special about it.

Methods *not* defined in `Object` have something special about them. They are rerouted to the "static" methods in [`RuntimeString`](https://github.com/scala-js/scala-js/blob/master/library/src/main/scala/scala/scalajs/runtime/RuntimeString.scala). (well, they will be when this portion is implemented)

Methods defined in `Object` are implemented by hand in the IR of `String` instead at https://github.com/scala-js/scala-js/blob/master/project/JavaLangString.scala
They have to be in the IR for `String` so that they handle overriding correctly.


I don't follow, why do these need escaping?

The rendering on github is identical.


Isn't the purpose of a `WeakHashMap` to avoid that kind of memory leak?


What changes does this make to the fields? Django sometimes makes migrations for things like label changes that don't really affect the database - those aren't really needed.


Ok, @rjmccall, @egranata, @jckarter, here is the patch, greatly reduced. A couple of things I'm wondering about taking this to the next stage - replacing the target-specific members with types based on `template <unsigned PointerSize> RuntimeTarget<PointerSize>`.

- Pointer-like members
  - Relative offsets: How to best capture getting the absolute pointer in the pointer width of interest
  - Pointers to other structures in the remote target: How to best provide the native pointer type to the local runtime vs the remote reader
  - Function pointers: These are never called by the remote reader, so these can probably be a `reinterpret_cast`?
- Any special considerations for `size_t` members?


OK, I'll investigate that.


As of today this still isn't fixed. The source code is correct but the web page has the following:

> If the alias has the same name as the aliased command, it is necessary to refix the call to the program with command in order to tell fish that the unction should not call itself, but rather a command with the same name. ailing to do so will cause infinite recursion bugs.


>make PR to make paper bins only have 2 high quality checks in their if() instead of 3 low quality ones
>new bug pops up that requires another check bumping the if() checks back up to 3 again

guess that's just how it goes


As far as I can see this lambda is only used once.  If that's right is it worthwhile?  You could implement the assignment to `trIn` using the ternary operator and `trIn` could still be const.  Just a thought.


Why not? Initializing a pointer is fine, as long as the pointer is known at compile time (i.e, a pointer to a global constant), and the length calculation can be constexpr if traits::length is.  In general, std::char_traits<T>::length is not constexpr, but you can use your own traits class.


Este commit no aporta valor.
abrazo


@sean-smith: The web page has the docs of the last release, so this'll be fixed once we make a release (which shouldn't be that long anymore).


From Magento's official github repo. You can check there and you will see there is nothing but an .htaccess inside the [vendor](https://github.com/magento/magento2/tree/develop/vendor) folder.


Why in the end did you not need this? (I was expecting it to be necessary)


This api specific code should be delegated the WeatherApi object. That way if we end up wanting to reuse the logic on multiple screens, or end up changing the way we interact with the api, we won't have to redo the code ( and inevitably miss something and break things).


partially fixed bugs in multiple foods' nutritional information
made plus/minus blobs bigger
bottom navigation bar changed
fixed suggestions (<select>) reverting to actual design


Done.

Thanks for letting me know.


Thanks!


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#UXWNUkyDR5-QfJRuGvbpfw


FWIW, I too am a fan of the `static` suggestion.


5499-Electric-Service-Follow-Up (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2124 of 2126 tests passed, 7 test warnings)

 * 9 tests had: AUD diffs.
 * 9 tests had: MDD diffs.
 * 9 tests had: MTD diffs.
 * 1 test had: ERR diffs.
 * 2 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: RDD diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.91%25-yellow.svg)


Why not an additional `r.pipe(log.newStream('readCache', size))` here?

The only new thing needed would be a stat on the file to determine its size.


Since `dr` and `cr` are not needed at the bottom of the function, I might be inclined to put them each in their own scope.  It would be really easy to type `dr` here and the compiler wouldn't catch the problem.  Just a suggestion.


In general, I would like to reduce, not increase, the number of events that are driven by polling.


looks good


siendote sincero, no entendia bien como trabajar con git. Creo q ahora, despues de vaaaarios tutoriales, ya entendi la onda.


A fixed upper bound based on `MAX_PLAYERS` multiplied by some constant would generate simpler code and would not greatly extend the loop even in small games.


"Help" is ok like this, it explains how to create tests

You can look in "History" (all the tests that have been done, in the last few years, but sometimes the code is missing)

or in "SF GitHub" and look in the Commits (all the tests which were accepted)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17503](https://teamcity.mesosphere.io/viewLog.html?buildId=147774&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:19


Would be good to comment what the returned `bool` tells the caller.


Works great! Thanks!


変更部分について確認しました。


さらに orb.finishCalibration() 後の ; が抜けてますね…
確認してプルリク投げなおします。ご確認ありがとうございます！


You'll have to change this part as it should work also without wallet.


missing `-H` option; + arguments aren't alphabetized


Looks good. However there is a general problem with changing article names and permalinks. @pblanz will create an issue for this.


missing options


captiveportallogin.apk force close how can I find a fix??
Help master??


ref: #2


How about to convert them to mdoc format ?


removing final needed to allow @InjectMocks in UnitTest


How to add custom service to add GUI interface ?


Not sure if serious, but given only 26 people have committed to `inference.jl`, no, about 95% of us haven't had such adventures.


false


roff => mdoc


Hi Catalini,

This change seems to break something. The desc.xml reports
```
<satip:X_SATIPCAP>DVBS2-0</satip:X_SATIPCAP>
```

instead of
```
<satip:X_SATIPCAP>DVBS2-2,DVBT-1</satip:X_SATIPCAP>
```
which would be correct with my netceiver setup.


You can remove these lines centering the cursor, the lib already does that!


dot


。。。我来看看


我也搞搞(●—●)


So, this forces the gamemode of the server.properties to apply to individual worlds that have had their gamemodes explicitly set?

This seems like a valid interpretation, but another would be that the per world gamemodes are force set to players logging into those worlds.

e.g. if someone logs out in survival in a creative world they will be set to creative instead of being whatever gamemode they were last or what the server.properties is set as.

If I have a multiverse setup that sets a world to creative, personally I would have expected force-gamemodes to respect that, as specific beats general. And only use the server.properties gamemode for unset gamemodes on worlds.


mk2latex => mkd2latex


license added


근데, 작성자가 아닌 오너가 `quantifiable/...`에 파일을 변경하거나 생성할 수 있을까?


please add a documentation comment


forgot checksums  ?


actually now it's with arguments you can test it, if you give it the executable `agroiy` or whatever it won't find it, right? Please include a test.


why not supply it as arg directly? This has no use


그건 안될거 같지...?


same


To fullfil our naming conventions we need to rename this function.


I don't like the bracket position, did the PEP8Bear really accept that!? Why! Though if it accepts that I guess I'll have to accept it too...


why not put it directly in the classmethod? I'm not a big fan of those useless variables, if you do proper breaks it's still well readable


ah thx for fixing


To fullfil our naming conventions we need to rename this function to `loop_wait_for_response()`


path -> dir


no other severities? Have you checked that?


To fulfill our naming conventions we need to rename this function to `loop_poll_events_until`


이 파일의 오너는 quantifiable이니까 오너는 모든 코멘트를 수정할 수 있는데, measurable은 quantifiable의 코멘트를 수정할 수 없는 거군...


`cmark_strbuf__initbuf`, `GH_BUF_INIT`, `BUFSIZE_MAX`, and `cmark_strbuf_init` should be kept internal.

We'll need a constructor function that returns a newly allocated `strbuf`:

    CMARK_EXPORT
    cmark_strbuf *cmark_strbuf_new(bufsize_t initial_size);

(Four outside users, `struct strbuf` should be an opaque type, so its internals can change without breaking the ABI.)


Слишком много изменений в одном коммите: добавлен новый функционал и изменено форматирование. Следовало разбить на два коммита.


@Annih do we still need this fork and this commit?


Heh, thank you for the hint. Now `fts.3` is included.


egeasg


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17613](https://teamcity.mesosphere.io/viewLog.html?buildId=148253&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:09


Неинформативный заголовок коммита.


acewafw


afawfaw


we could, what's the point?


`cmark_strbuf_free` should be renamed globally to something like `cmark_buffer_release`. Then we'll need a new version of `cmark_strbuf_free` that clears the buffer *and* frees `struct cmark_strbuf` itself. (Internally, we always allocate `struct cmark_strbuf` on the stack. This is not possible for outside users without exposing the struct's internals.)


您好，我是一个php新手，下载了您的实例后，我发现并不能正常登陆进去，我已经修改了user表中的用户名和密码，期待您的解答！


Cambio


@labrys I know this part is pretty ugly, but it was the only way to get it to work. Been messing around with it for 2,5 hours. if you can come up with a better solution, be my guest!


@ravinimmi you had the issue, does this fix it?


@pymedusa/contributors I know it works for when we move from no results in cache, to results. But i'd like to see this tested,, with updated results. Like when seeders/leechers update.


@sils1297 Yes it does.


解决了，依然感谢！


Suggested documentation: Returns the character data as null-terminated string and clears the string buffer. The returned string must be freed by the caller.


I think this function calculates radon transform for zero degree instead of 180 degree since it integrates along vertical lines for each column on the source image from left to right and it writes (sum) result to destination image from bottom to top for theta=180. You reverse y axis when you use 180 degree on radon, so your xp value (distance between origin and integrating line) starts from positive value instead of negative. So I think it should write (sum) values from top to bottom to destination image. I commented out this line and it seemed smoother to me :)


This commit cause #3 and is solved by #4


merging then, @AbdealiJK may file a bug if he wants that


\+ markdown.3 contains some [typos](https://github.com/faelys/libsoldout/compare/84c979675cc1ac22b22b27644bf4c23b7ac58f08...trunk#files_bucket) (with fossil it would be better, to show diff only for the one file...) and some markdownizm


Not really equivalent to `strchr` because `strchr` returns a `char *`.


Восемь человек в один коммит под одной учеткой — все-таки перебор.


Documentation should make clear that this returns the number of bytes, not UTF-8 characters.


Should also provide English version


Also compares strings lexicographically and returns a negative value if a < b and a positive value if a > b.


Этого не следовало коммитить.


Заебал баги плодить)))


If all the scrapers right now uses the runtime value maybe you could fill it with the runtime of the last episode..


You can implement Haxe break as native goto to avoid that.


Drops bytes not characters.


Oh hast schon angefangen! Bist du nun doch teils ins Deutsche über gegangen? -'lotgeraden'-system - Ich glaube man könnte es auch perpendicular nennen. Heißt übersetzt Lot.


YAY! Should I type "red" for example?


В списке появились повторы. Следствие кривого разрешения конфликта.


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.5 [Build 113](https://teamcity.labkey.org/viewLog.html?buildId=306018&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 11 (1 new), passed: 39 Build time: 01:17:30
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

org.labkey.test.tests.CDSSecurityTest.verifyUserPermissions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyLogGutterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisFilter: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyProductsItem: <no details avaliable>

##### there are 1 more failed tests, see build details
```


Truncates `buf` to `len` bytes if its size is greater than `len`.


we are friendly.


@mikaabra
I realised creating your device as a BDS may not work, as requests are slightly differents beetween BDS and AVR. You should test the sleep action from the forked repo, but creating your device as an AVR, like in the exemple.php file.
I just created the pool request, so if Karim merge it, you'll be able to fallback on the main repository to go on with your devs.

Please, let us know if the sleep ation runs as expected on your device.


When you through a new release, you could use `git tag` and Github's release feature.

See more on https://git-scm.com/book/zh/v2/Git-%E5%9F%BA%E7%A1%80-%E6%89%93%E6%A0%87%E7%AD%BE


I'm using the sample image and I get an exception on this line. `x` value varies from 1 to 200 in loop and when it's 200, it tries to get pixel value with `src.at<T>(0, 200)` command for 200x200 image. I've used `x-1` as a workaround for this but there may be something I do wrong, can you have a look at this, please?


... and I don't think it's more of a security hole than @ syntax in general, which is why it was removed in 5.5 and above.


comment to comments


Should be `GP_Translation`


Not wanting to put my head in the lion's mouth again over this, but can we please now finally drop 5.4 support?


Either solution, merging or keeping it separate is better than this one.


Thank you!


Isn't this `GP_Validation_Rules`?


@jethrokuan, it's actually like what @bucaran had said, someone might have no support for UTF-8, only some terminals do (framebuffer will do too).


Aha, OK!
Thank you both.
Richard


Ohh yeah. That is again an indexing mistake. Thanks again. It should be  `x-1`


can you also add a test with an escaped quote?


@jethrokuan Some man will not display it. Too bad.


@pickfire This is already merged in my local copy. I will be pushing your commits with my new modifications later tonight.


ERROR: 'tumeKyouenModel' hides a field.


ERROR: 'context' hides a field.


WARNING: [SmallSp] Avoid using sizes smaller than `12sp`: `10sp`
 Avoid using sizes smaller than 12sp.


I dont see issues with dvb adapters, I was wondering make clean;make helps ?

I've added some fields in adapter structure.


Oh, schön, er hat einen Namen bekommen UND bei beiden Knoten wurden die Genkoordinaten korrigiert!


Fixes #6


Fixes #7


I agree, but adding the plumbing to do this is not trivial (there's been at least one failed pr already), and entirely separate from these filesystem-focused changes.


I am not in the physical location of the device either, I'll test your suggestion next time I am there. Right now my main goal is to defeat the 8 hour idle go-to-sleep feature, because I want the receiver to be on at all times. Right now I am issuing "on" every hour plus changing the input, waiting 5 seconds, then turning the input back to where I want it to be sure. Do you guys have experience if just sending "on" to the device will keep it out of idle mode?


Why?


Archivo mencionado*


Seems the CI needs an update to compile this. The recent nightly build failed. Can someone takena look?


Aha! God pointe. Lad os holde os til LTS. (Ikke at det forhindrer mig i at future-proofe koden frem mod 1.11 LTS i april 2017.)


delete


`bufsize_t` should be renamed to `cmark_bufsize_t` in `cmark.h`. In `buffer.h`, we can add

    #define bufsize_t cmark_bufsize_t

so we can keep using `bufsize_t` in the cmark codebase.


등록된 사용자가 3명 이상이면 어떻게 하나...?


sure :)


Good job!


Is the option 'neverReplaceExistingPlugins' false in https://jenkins.ci.cloudbees.com/job/core/job/acceptance-test-harness-stable?
I added a new test that should have been skipped (analysis core 1.76 is not yet released), but actually it has been exercised: https://jenkins.ci.cloudbees.com/job/core/job/acceptance-test-harness-stable/lastCompletedBuild/testReport/plugins/WarningsPluginTest/should_resolve_workspace_files/


@haslinger Sorry, trying to catch up. I'm fine with true, or just a success code would do as well.


See https://github.com/jenkinsci/acceptance-test-harness/commit/cee24aa85bec9ca281ba2762459d17e0cbe1da2b#diff-0ec2f46a23af83f72bfa0ee14b8c9104R77


あ、srcに気を取られていて、「配布物云々」も原文にないことには気付いていませんでした (^_^;)


Typo fail :) "model"


@haslinger No hurries; 200 OK alongside, obviously.


There's no Plistwriter in the standard library..?


YES! This.


*now