I don't like the parameter name. Don't include "flag". Also the name and description won't make sense to users. Maybe "resolve_peptides: Post-process Fido results to resolve peptide/protein associations using a greedy Occam's razor approach"?


This ErrorPublisher trick is in order to be able to nicely support `() => new FileInputStream` which propagates the FileNotFound on materialization.


I agree. See #10742.


Can you at least use `beforeEach` instead of `before` then? Anyways, let's not worry about this too much. If you think the current version in this PR is the best solution for now, then I am ok with it.


close #3


Ok so the problem that gets fixed here is that if you wanted to rename variables then you only looked at the children since you were traversing the AST.  init cond and incr were ignored since they were not considered to be child nodes


Other fields have this issue too, e.g., authors often have multiple lines in my bibtex entries.


is this whitespace necessary ?


space before "condiment"
"your $rnd(favorite,least favorite)" can be replaced with "your$rnd( , least )favorite". with spaces ofc


Do we perhaps want to do in this direction?
// Form a user's perspective I'd like a "file source", without having to care about FileInputStream.


-ECANNOTPARSE


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14047/testReport/) for this commit**


Change "it" to start with capital


Comments in the code are not a good medium to have such discussions. But I agree with your argumentation, so just remove the comments.


whoah, this line is wider than https://github.com/lallinuo's stack.


Before go ahead with merging this pull-request I would like to make sure that we are actually capturing what we are supposed to be capturing here.

For example,

1. Do we need to filter out operators with 0 length (is this a valid Cigar at all?)
2. Do we care if the cigar has consecutive operators of the same kind? what we should fail for two consecutive insertions (xIyI were x and y are numbers) but not for consecutive match (xMyM).
3. Why we should not allow for a Insertion to be followed by a Deletion or viceversa. I think it is a genuine possibility otherwise you are kind of forcing an artifactual base map. Perhaps I am missing something here though.
...


Who is managing the locks dictionary (iex. Clearing it?)


more about problem:
https://www.dropbox.com/s/jcmssxwcrsfpla4/Zrzut%20ekranu%202015-04-21%2012.46.38.png?dl=0


@ajinabraham its showing an error: Cannot Reach : http://192.168.1.1, do u know what to do


Voll gut. Gefällt mir, Joshua.


Sorry I add an error in line 104 (Hate copy paste)...


yep new package with something like jenkins_ci_support would be great, maybe add there own rake task for ci integration? I do not like to have it everywhere as it is not for all developers.


Actually, a different approach, like:
```
[["label1", "label2", "label3"], ["set1val1", "set1val2", "set1val3"], ["set2val1", "set2val2", "set2val3"], […]]
```
could be used, but it would require a bit different GUI.


coding style: no capitals in variable names (except abbreviations) - see https://github.com/OpenMS/OpenMS/wiki/Coding-conventions#variable-names
(applies to all camelCase variable names!)


To tu jest tak że oba się do siebie zbliżają? to bez sensu; jeżeli `A` ma większą intensywność to tylko w `fireflyB` należałoby ustawić `JobsDistribution` na `children[0]` (czyli na crossover w ~40% podobny do lepszego, myślę).


@dchambers should this be something more specific (e.g. `brjs-html-templates`) so theres much less chance it'll clash with other IDs?


Good Work ;)


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14048/testReport/) for this commit**


coding style: only break line here if it would be longer than 80 characters, otherwise prefer to break at ";"


I haven't thought this through:

How about write into tempfile and then do a file.copy with override?


@khadijaibrahim  @samaasallam


Could you change this to a string? Also might be worth adding ``FB_APP_ID = ''`` into base.py.


coding style: remove line break after ";"


@terminalmage that means that if both `keyid` and `keyid_ppa` is `False`, remove_key will still be True?


Nice!


If the intention is to port what there is in GATK3 as it is for now, the reg-expression provided is not doing it fully.

For example GATK3's filter would not complain for consecutive H operations at the beginning whereas the new code does.

Also it does accept non empty CIGAR with out any operator that consume read bases such as N or P. That would not happen with the GATK3 code.

I don't know if this can be captured appropriately with regular expression.


Oops, thanks for spotting! Will fix.


Yep, good call! Thanks @andyberry88.


Hi,
what do you understand under "hidden" TeamSpeak directories?

What exactly does the code ´-not -path '*/\.*'´ mean? This means "anything/<hidden_files>" or?


Yes, randomization of the tests to increase coverage is an excellent direction.


Can't find this release on maven central.


> Do I need to init Sets?

No. Map elements are default-constructed when you access them via "operator[]" and they don't exist yet (see http://en.cppreference.com/w/cpp/container/map/operator_at).


test


sadf


It might work but the objective is to update the formula to use the new repo.

I had opened a pull request for this but closed it after in favor of another one which was already opened.


Boo forgot to tag with issue number #83 Sound toggle now working for question modal


Sorry to say that, but this code is messy...


This should be 0.20.205.0, right?


Small typo. Should be:

    $ python3 Sieve_of_E.py 1000 prime.dat


Excuse me, can I ask, why did you decide to delete this command?


now we must modify the quest for branches >=3.x :D


This is wrong and should be reverted. OS X 10.10 has kernel limits of 12k files total.

    Mac:mongodb bmike$ sysctl kern.maxfiles kern.maxfilesperproc
    kern.maxfiles: 12288
    kern.maxfilesperproc: 10240

Unless you're going to test for non-default kernel settings or explain how to raise them, it's better to just bump mongodb to 1k files open and not have everyone mucking with kernel limits.

I've been running at 1000 soft files and 1023 hard files with no ill effects for several months and submitted a PR to basically revert this. https://github.com/Homebrew/homebrew/pull/38871/files#diff-21398193c1e6dbab49264de41589c9f1


fine with me, I'll forward any "where is my 7sec smallstepback" shitstorm to you ;)
You'd also like to remove 15m? I'd probably also only disable it because I'd like to be somewhat MediaPortal compatible to ease transition.


committing changes into branch


Shouldn't you be looking for hadoop-common instead of hadoop-core?
http://mvnrepository.com/search?q=hadoop


Fixes #1


GET should not be used for an operation with side effects such as this.  It should be turned into POST instead.


tTimerEventArg structure contains the tTimerHdl and the timer-generic module uses them as the event argument in kEventTypeTimer events as shown below.
![2015-04-21 16_34_14-openpowerlink - microsoft visual studio express 2013 for windows desktop admini](https://cloud.githubusercontent.com/assets/2918478/7250975/506624c6-e844-11e4-9189-53e881e6809f.png)
 Timer events which are posted from user to kernel then creates the problem explained above.


Okay.


thx @vBm for fixing typos


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14050/testReport/) for this commit**


Yeah, well there is that but you can mine a lot before you have to clean out your inventory and if you fill up all the inv slots with at least 1 good material the cobble won't fill empty slots. At least that's Eats system.


I think this can be left away. The update logic is a bit messy, but it should be sufficient in this case. The update is timed (to prevent updating the properties too often), but if the cache goes idle (i.e. EOF) before the next update, the next update will still be carried out.


Probably not necessary.


Wouldn't it be better to return unavailable?


Changing


read-me is not ready!


If we stick to using regex ...

I don't see the need of the of() operation at all with an or(String ... comps) that would join using the sep "I" and surround each component with brackets makes the code more readable IMO. Also you would be able to get rid of the clutter from those repetitive string concats and pipe ("I") string literals.

You could have static method to add starts or crosses star(), plus().


Awesome!


I think that pat should be renamed to something more informative and using upper cases and underscores ... E.g. BAD_CIGARS?


Oh right, I missed that one...


I have tested it on my Fedora Notebook and in the virtualbox vms with CentOS. I didn't try it with Ubuntu...


@keradus @gharlan

Nice, this build fails with this:

```
$ php php-cs-fixer --diff --dry-run -v fix
Loaded config from "/home/travis/build/FriendsOfPHP/PHP-CS-Fixer/.php_cs.dist"
....................................................................................................................................................................................................................................................I
Legend: ?-unknown, I-invalid file syntax, file ignored, S-Skipped, .-no changes, F-fixed, E-error
Fixed all files in 78.153 seconds, 18.010 MB memory used
Files that were not fixed due to errors reported during linting before fixing:
   1) build/logs/clover.xml
The command "php php-cs-fixer --diff --dry-run -v fix" exited with 3.
```

:question: Why would one even attempt to fix an XML file?

See https://travis-ci.org/FriendsOfPHP/PHP-CS-Fixer/jobs/59381241#L236.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14051/testReport/) for this commit**


How about "<npcname> succumbs to the infection"?


test


Le look du cube est pas mal :D


Oneshot repaired this card by using just the number as a single parameter to SetCountLimit


Please rename this function to 'scheduleReconnect' and we're good to go. :-)


Finish with my review back to @akiezun


There's a typo in the filename "Triert_Uni.json" (should probably be "Trier_Uni.json").


The idea is to remove the hard dependency. The AFN request/response serialisers we can use directly by including the subspec, no point reinventing the wheel.

One option would be having a default implementation of HTTPClient that works on iOS7+ only (NSURLSession). If you want iOS 6 support you add AFN as a dependency, and have rest kit auto detect the presence of AFN and use an AFN specific version of HTTPClient. Thoughts?


LaTeX3 语法下空格是被忽略的字符，所以无所谓。


lol


@petrandreev, why did you add these three methods? Something very similar is included in AbstractPopupMenu.AdvancedPopupMenuInteractions


We've decided to switch from supported to provided, which better describes our intents.


Hi, I stumbled upon this change while investigating a bug we experienced using PhantomJSDriver.
To my understanding the default semantic to use here should be "quit()"
 instead of "close()".
close() only closes the current open window, quit() closes all windows and dispose the driver resources.
Is there any reason why we kept close() as default?

Reference to selenium ML:   https://groups.google.com/forum/#!topic/selenium-developers/SjeNBn1Ssd8


ups hab ich das committed? habs eigentlich excluded.


I think some assemblies are missing when packaging the zip files.


Any thoughts already on a maintenance release for 3.1.x that would include this fix?


If I remember correctly it did not which makes sense as the reply would be a subclass of Error and not an instance. I'm kind of misusing classes as constants there.


That yields a 404 on `opensource.org` :)


Hi @01J ! There is no need to keep logs in git repo. You should use .gitignore file to not commit them.


Why don't you use the ternary operator?


That yields a 404 on `opensource.org`


See my previous comment re supported vs provided.


@k-joseph did you intentionally next the spring bind path of command.concept.displayPrecision under command.concept.precise?


Thank you for using us as a reference for your kernel :)


Someone ought to have mentioned that before, but sure


You could use

```json
"contao-components/contao": "^3.4.3",
```

here.


Could this be moved into `FORMField`? Would that be a good idea?


Hey,
In Unix systems files and folders starting with a dot are considered hidden.
http://en.wikipedia.org/wiki/Hidden_file_and_hidden_directory#Unix_and_Unix-like_environments

So if you have some TS3 backups and/or Test folders you put them - for example - in /home/ts3/.backup
The ".backup" folder is hidden.

The added find option means exclude all paths that somewehere are hidden (have a leading dot)
(-path glob/regex)


Maybe we could have a category `FORMField+ FORMFieldValuesTableViewController`


What do you mean?


It's from autocomplete_light v1 and wasn't used anymore.


remove TTL1 action from switch attribute


The build team has reviewed. Please let us know if this needs to be back-ported.


What is this supposed to do?


What is this `rescue` line supposed to do?


No, I'm not complaining about having the _edit pedigree_ button in view mode, I'm complaining that the `/view/` action is used when editing the pedigree.


If used like that you are right, it doesn't. Ignore me then :wink:


@k-joseph  FWIW, that change caused TRUNK-4669
So i have changed it as: https://github.com/openmrs/openmrs-core/commit/92a3cf4df17409bf04565618511603ee613ed025
Feel free to comment if this reverts what you were trying to fix.


Reverted and added tests


it's work if your existing fmd is older than the latest one. Try to download older version and you will get updates dialog with changelog.


@mouchtaris This code was pulled from the codebase in `fog`. If you find an issue on it, please submit it [here](https://github.com/fog/fog-google/issues).

This commit is to big and has nothing to do with the actual code so comments here will get lost. =)


default=10000 or default=models.BigIntegerField.MAX_BIGINT


Lev has more than 14 years of experience in the software industry (Israel Electric Company, Advantech, IDC). Lev's expertise is Web Development of multidisciplinary information systems.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14052/testReport/) for this commit**


这个div节点可以省掉，直接用<ul>就足够了，尽量避免冗余的节点


I understand. It will be fixed.

From: Michal Kwiatkowski [mailto:notifications@github.com]
Sent: Tuesday, April 21, 2015 1:49 AM
To: Mei152/Bloccit
Cc: Mei152
Subject: Re: [Bloccit] Complete crud-assignment (bdc6e47)

You changed this line and that change was unnecessary. That was a blank line and you've added two spaces here.

—
Reply to this email directly or view it on GitHub <https://github.com/Mei152/Bloccit/commit/bdc6e47197f3b97aa9534f9ce46f897c114d65fa#commitcomment-10817426> .  <https://github.com/notifications/beacon/AKgq0CJ7Ulk3UscoFSIIN5cSxCnOg1Vzks5oBdxTgaJpZM4EEIEf.gif>


May I suggest:
+  2 * popcount<Max15>(ei.kingRing[Us] & pos.pieces(Us, PAWN, KNIGHT) & ei.attackedBy[Them][ALL_PIECES])


COMMIT!


AA


I haven't dug in, and I don't know what has changed, but GeoServerIT does seem to "randomly" fail more than half the time these days.


尽量合并为font简写


Hey,

thanks for feedback. We do not expect large number of keys there, but I have applied following fix to decrease the potential memory consumption even more: https://github.com/ayende/ravendb/pull/2145

It is roughly 7MB of memory for 100000 entries (indexes) now.


See my previous comment re supported vs provided.


Nothing should clear it.
It isn't going to hold a large number of values anyway.

*Hibernating Rhinos Ltd  *

Oren Eini* l CEO l *Mobile: + 972-52-548-6969

Office: +972-4-622-7811 *l *Fax: +972-153-4-622-7811

On Tue, Apr 21, 2015 at 1:47 PM, Daniel Marbach <notifications@github.com>
wrote:

> Who is managing the locks dictionary (iex. Clearing it?)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ayende/ravendb/commit/c3a827cd19b79dc6075a977e423b8d68e37ab204#commitcomment-10821806>
> .
>


See http://stackoverflow.com/a/4137582/674064 for how you can make the tests pass again when relying on `request.META['HTTP_HOST']` to be present.


I don't think you're supposed to do `this super`.

If you can't use `this _count`, can you also not use `this count`?


There are some colors that you have missed.
example that "functions" is blue.
And you haven't added the red color on the border of important words.

![skarmavbild 2015-04-21 kl 13 42 25](https://cloud.githubusercontent.com/assets/11715450/7251503/56743be0-e82c-11e4-8428-f87b1d209ce1.png)


Thanks for the immediate response. I have submitted already an [issue](https://github.com/fog/fog-google/issues/20).


test


__API Blueprint syntax error on line 3276__: Undefined resource model 1039 (Autogenerated message from apiary.io)


I'd prefer that each and every single image has identical headings, this way it's easier both for us to find what's needed/missing and for reader to navigate through the doc.


I copied "this super" from FloatVector. Shall I change that to "super"?

I can assign "this _count". Just read. But I can assign "this count".


Note-to-self: only use decimals for chrome extension...


This is hilarious


I know what Six does. My question is about Six is not referenced in setup.py and there is no comment in code about it.


Hi, This is a looking good one plugin ! I'm a windows user and i wanted to know if it's compatible with windows users.
i ve download from the repositery and i ve  rename the folder  from "sublime-PHP_CodeSniffer-master" to "PHP_CodeSniffer" then put him in \Roaming\Sublim text 3\Packages\
i have edited my settings files with this value :
// Path to PHP.
    "php_path": "C:\\Program Files (x86)\\EasyPHP\\binaries\\php\\php_runningversion\\php.exe",

// Path to the PHPCS script.
   "phpcs_path": "C:\\php\\pear\\phpcs.bat",

// Path to the PHPCBF script.
 "phpcbf_path": "C:\\php\\pear\\phpcbf.bat",

I've got this error in sublim text console :

@echo off
REM PHP_CodeSniffer tokenizes PHP code and detects violations of a
REM defined set of coding standards.
REM
REM PHP version 5
REM
REM @category  PHP
REM @package   PHP_CodeSniffer
REM @author    Greg Sherwood <gsherwood@squiz.net>
REM @author    Marc McIntyre <mmcintyre@squiz.net>
REM @copyright 2006-2014 Squiz Pty Ltd (ABN 77 084 670 600)
REM @license   https://github.com/squizlabs/PHP_CodeSniffer/blob/master/licence.txt BSD Licence
REM @link      http://pear.php.net/package/PHP_CodeSniffer

if "%PHPBIN%" == "" set PHPBIN=C:\Program Files (x86)\EasyPHP\binaries\php\php_runningversion\php.exe
if not exist "%PHPBIN%" if "%PHP_PEAR_PHP_BIN%" neq "" goto USE_PEAR_PATH
GOTO RUN
:USE_PEAR_PATH
set PHPBIN=%PHP_PEAR_PHP_BIN%
:RUN
"%PHPBIN%" "C:\php\pear\phpcs" %*

Then i have create windows environment variable
PHPBIN with value `C:\Program Files (x86)\EasyPHP\binaries\php\php_runningversion\php.exe
And change :

// Path to PHP.
"php_path": "%PHPBIN%",

the error message in sublime text disappeared
but ! no error is detect in my php file :( seem to be not working.
Do you have a working PHP_CodeSniffer.sublime-settings --User Example ? Thx for help.

Sorry for posting here. I know it's not the best place but it's difficult to contact you. I will delete this post if you ask me. And sry for my english :)
Cheers. Thx in advance.


Ah, it's a qml plugin you running under lipstick? Then you need to send call to daemon, and execute desktop file from it.


Thank you for the concern. It should be fixed now.


You don't want this.


Damn.. not quite :( I cannot just use the current date with getOffset, I need the date that was clicked in the schedule. Back to the drawing board....


Make return types less specific where possible. In here you can return List or maybe even Iterable :)


You are welcome. Thank you for the feedback! :heart:


`ret` is always false at this point. can we completely remove `ret` from this method?


html中为ul定义了class名‘list-h’，就直接使用类选择器


В принципе можно было бы и перевести вкладку эту, т.к. там не название мода)
Но, да ладно. Пусть будет так :)


This is actually already tested here https://github.com/newrelic/centurion/blob/master/spec/docker_server_spec.rb#L69-L75


CI says this variable is unused.


No, you don't want to add RequireJsParts.


if am reading this correctly, if angular.isObject(object) return true, object will have a Boolean value instead of Object.
```Javascript
if(!angular.isObject(object)) {
    return false;
}
return (Object.keys(object)).length === 0 ;
```


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14053/testReport/) for this commit**


@yairwein Because that is our server address naming convention?


这里的图片是为链接修饰样式，可省去img标签，设为a标签的background


Hi Pavol, I didn`t want to modify the AbstractPopupMenu since DropDownMenus extend it and are generally static- so there is usually no need to inspect their labels or values. The "dynamic" contextMenu implies the re-rendering of MenuItems therefore it is necessary to examine, f.i., the newly rendered MenuItem label to match the selected row data before triggering the AJAX action (otherwise the ContextMeu would immediately disappear). The AbstractPopupMenu implements  selectItem methods in following way: 1st the Popup is ALWAYS shown and 2nd the item picked is selected. It is essential for the dynamic contextMenu use case first to show the popup, analyse the items value/label and then select some visible item for the (AJAX) action (and the tester can see it while using for example chrome/FF driver).


__API Blueprint syntax error on line 3204__: Undefined resource model 1039 (Autogenerated message from apiary.io)


使用简写


Bugfix on tag `0.2.1`. Thanks @dgud!


To leave this package as simple as possible and eventually move the commands and other stuff to a separate package.


implement*


I think the 'In' is superfluous here.


Maybe 'please refer...' :)


Nevermind :) Just saw that we have other dependencies like `customHeight`


I would use sometihng like 'loss function expression' here, to make it clear that we're talking about a Theano expression here, and not a Python function.


Wouldn't https://github.com/ayende/ravendb/commit/c3a827cd19b79dc6075a977e423b8d68e37ab204#commitcomment-10821868 make it idempotent and safe by default? IO is already there so having a bit more IO wouldn't hurt.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14054/testReport/) for this commit**


为外层容器加上最小宽度


What Wildcard said.  PreAbility is fine-I should edit this to reflect that.


Your office was locked so I came to the photometry unconference. I'm not
planning on going to any of the 3pm sessions so maybe I can pick up where
you leave off from 3-4 (maybe even from 4-5, I can't remember what the
session is).

On Mon, Apr 20, 2015 at 11:17 PM, Fred Moolekamp <fred.moolekamp@gmail.com>
wrote:

> Sure, I'll do it tomorrow
> On Apr 20, 2015 5:12 PM, "Víctor Terrón" <notifications@github.com> wrote:
>
>> @fred3m <https://github.com/fred3m>, would you like to finish this
>> function?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/vterron/astrometry-wrapper/commit/a91c3e5e7f8ebd5c22c8e10085faee12ae603ce7#commitcomment-10805268>
>> .
>>
>


Кстати, имя вкладки так и осталось переведённым ;D
By the way, tab name isn't fixed :D


The example looks great! I would use `x` and y` here instead of `sym_x` and `sym_y`. I guess the `sym` prefix is for "symbolic"? I don't think it's necessary at any rate.


Hi @edds - thanks for your reply and taking time to explain this technique; it makes sense.  Caution of tone noted, thanks.


return?


这里标签使用太多，可以简化，最多两个div即可，图片作为外层容器背景居中


No, it wouldn't.
It would just mean that you moved the issue to the move of temp to file.

*Hibernating Rhinos Ltd  *

Oren Eini* l CEO l *Mobile: + 972-52-548-6969

Office: +972-4-622-7811 *l *Fax: +972-153-4-622-7811

On Tue, Apr 21, 2015 at 3:02 PM, Daniel Marbach <notifications@github.com>
wrote:

> Wouldn't c3a827c#commitcomment-10821868
> <https://github.com/ayende/ravendb/commit/c3a827cd19b79dc6075a977e423b8d68e37ab204#commitcomment-10821868>
> make it idempotent and safe by default? IO is already there so having a bit
> more IO wouldn't hurt.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ayende/ravendb/commit/c3a827cd19b79dc6075a977e423b8d68e37ab204#commitcomment-10822896>
> .
>


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


why did you remove icon_id, now it breaks for us?


I've left the pointer as a finger for two reasons:
1) As an unobtrusive reminder that Quick Play is available in the timeline, even if Tooltips are off.
2) Because the right click menu is still available.

I prefer that if the user wants to use the Quick Play feature, they get the full-featured Quick Play rather than reverting to the old version that you previously described as inadequate. If the user does not want to use the feature then I agree that they should be able to turn it off so as not to be distracted by it. I do not want to make the right click menu overly long, so I am reluctant to add more options, at least at this stage.


agree. will merge this.


muy puro


@tcrsavage can you have a look at it and suggest anything that I can improve or add :) Thank you!


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14055/testReport/) for this commit**


typo: n moved from exception to path


Ok, i've optimised searching the texture cache:
- Look for the texture by address 1st, this is the common case that happens almost all the time
- Don't do anything different when the texture is not fully hashed
- Reduced search loop to one if statement
- Changed (tex_hash ^ tlut_hash) to full_hash. Not sure if the compiler optimises something like this, but doing the same 100 times in a loop seemed like a bad idea

So in short, only if a texture is not found in the cache by address, the whole texture cache is searched. So the basic problem remains, and the worst case is still searching n entries n times each frame. But if a texture is not found in the cache, that's the general worst case already anyways. I'm not sure if this is really a big problem.

One other problem on top of that is that if a texture is not at the same address, it will be found in the cache, but Dolphin will always do the long search for it. Is there a fast way to clone a texture cache entry? Or a good way to link from one to another?


Like where this is going but dosent feel completed. Currentlly look like a cutted diamond its missing the toolbox felling to it.


@anthonycrumley `Thread#status` is global per-thread and it might be `sleep` before actually waiting on the queue, for instance because it waits a bit on another Mutex to internally access the queue data. `@num_waiting` might not be updated yet as that is incremented a few instructions later.


banner部分div使用太过冗余，整个banner使用3层div应该就足够，图片作为最外层div的背景居中，中间div设定980px宽度，最内层div即包含右下角的文本内容。


if {} else {}; branches must be compound statements


:smiley: :smiley: :star2:


标题使用h1~h6语义化标签


remove spaces at end of this line


Just finished tracking a bug: I believe this revision change is breaking my deployment to heroku due to the release download not yet being available.  Am I incorrect?

The url heroku tried downloading:
```
https://github.com/sass/node-sass/releases/download/v3.0.0-beta.6/linux-x64-14_binding.node
```

The url I found manually via browsing the releases page:
```
https://github.com/sass/node-sass/releases/download/v3.0.0-beta.5/linux-x64-14_binding.node
```

The only difference is ...beta.5/linux... vs ...beta.6/linux...


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14056/testReport/) for this commit**


段落建议使用p标签


Solved! :)


`last` should be last of results, not last inserted id.

```php
$db = new Database();

$usersResult = $db->read('users')->exec();

// First user from query result.
echo '<pre>', print_r($usersResult->first()), '</pre>';

// Last user from query result.
echo '<pre>', print_r($usersResult->last()), '</pre>';
```


or should we use the factory here? // if yes, there are other parts in the InMemory implementation to change


Thank you!


gotta flip this if statement.

instead of doing

if not something:
   x
else:
  y

it's more readable to flip them and remove the not, so

if something
   y
else
  x


this is not needed


Shadow's missing :)


newline


@julianrichen Any particular reason you didn't choose protected/private?


Curious. How exactly are you installing node-sass? There is no 3.0.0-beta.6 tag yet so you should not be getting this changes unless you're installing from git.


not consistent with comment in the code
this shows why you should remove the comments from the code


Warum die Umstellung? War unsere Lösung schlecht?


Should probably preg_quote here to ensure regex isn't broken by special chars


use factory? see above


yeah, right. Stupid me. Just verified

```
System.IO.File.WriteAllText("foo.txt", "Hello oren!");
            var tasks = Enumerable.Range(1, 1000)
                .Select(i => Task.Run(() =>
                {
                    System.IO.File.Copy("foo.txt", "bar.txt", true);
                }))
                .ToArray();

            Task.WaitAll(tasks);
````

Sorry for interrupting your flow of awesomeness :)


I was working inside — you should have knocked! Yes, I'm attending one of the sessions at 3 PM. I'll upload the changes I am working on before that time. Talk to you now.


Could someone want to filter the title even though none has been set via the shortcode atts?


> You'd also like to remove 15m?

drop it as we don't need it if we use 10 sec.


"15" should probably go in a constant at the top

so then

self.player.set_property('buffer-duration', BUFFER_DURATION * Gst.SECOND)


few nitpicks, but looks good otherwise. Let me test it


not needed


more coverage here, http://arstechnica.com/security/2015/04/1500-ios-apps-have-https-crippling-bug-is-one-of-them-on-your-device/


@jgoux - These changes are part of the JSPM configurations... but I will investigate impacts on direct npm installs.


lol re


And the migration?


I've changed this to

expect(invalid_vote.valid?).to eq(false)


ok merge


@frankcash  :+1: It was a pointless commit, I just wanted to join the #milehighclub. #ashamedofmyself


Maybe a good idea to change the declaration to ```static BOOL TestStream_Reading(void)```


you also dropped 15 min in your listing above (dropping 15 sec is fine).


If I were to do it my way, I'm guessing I'd have to iterate through the votes that belong to @post and check each of them to make sure they have a value of 1 or -1. I'd probably want to abstract that functionality out into a test_helper.rb file and include that back in.

Let me know if that sounds right!


Hello,

If I understand correctly this change, you set last used buffer to the buffer that was edited rather than the one that we are switching to.

I am wondering if this is desired behaviour, note that the 'ga' command switches either to the front of the buffer list, or (if it is already the current buffer) to the next one.
Do you get more predictible alternate buffer behaviour with that change ? or do you have the basically the same as before ?


Please also check that `findmydevice.logged-in` is set to `true` here.


And `false` here.


@cal101 well undestud you, but you argument is invalid


Same here, please assert that `findmydevice.logged-in` gets modified even when FMD is disabled.


@khadijaibrahim  @samaasallam


No, it's not. Might be an automatic import of Eclipse for some code which isn't used anymore.
But I'll test that...


Sorry!, I missed it, solved now too


"GetFor" does not seem like a very clear name (get what for what?). Why not simply call it "allocate"?


这种方式处理存在兼容性问题，也不可取，直接绝对定位即可


My apologies, I was unaware of how releases were handled.  I needed v3 and was referencing master directly hence the error.

I will raise an issue next time too, instead of commenting on the commit directly.

Thanks for the quick response.


The library returns RGB, not BGR. In addition the numpy array _npy_frame seems not to be compatible with the recording part. Thus I changed
```
@property
def rgb(self):
    return self._npy_frame
```
to the current code, allowing to record the world video.


In this commit https://github.com/jlapeyre/SJulia/commit/cc9fa3b7679df9fbea3f7d5e094ce5f0aadec988, I changed the key for entering SJulia to `=`. I changed the prompt color from brown (same as in Cxx.jl) to blue.


Is this merely for Carthage compatibility? Does the SQLCipher target clobber the SQLite target?


Wasn't the plan to add this to the core?


do zmieniania rozmiaru czcionki służy css


I think that's more than you need. If you create one upvote and then check the value of post.votes.first, then do the same for a downvote you'll have all the bases covered.

> On Apr 21, 2015, at 08:24, Piper Gragg <notifications@github.com> wrote:
>
> If I were to do it my way, I'm guessing I'd have to iterate through the votes that belong to @post and check each of them to make sure they have a value of 1 or -1. I'd probably want to abstract that functionality out into a test_helper.rb file and include that back in.
>
> Let me know if that sounds right!
>
> —
> Reply to this email directly or view it on GitHub.
>


let's jump over to the PR https://github.com/xbmc/xbmc/pull/6998


@javathunderman: Why don't you just use the 4-clause BSD license?


Ah right... I forgot how delite/register works.   `Bar._ctor.prototype.baseClass` seems to work.


It would be rather surprising if it didn't, no?


Ah right that'll always end in pain. I suggest `npm install node-sass@beta` for now :smile:


I think it's time to some auto update for default-bootstrap without need to update entire core because things like this are critical :)


@andreasohlund I'll be your memory here ;) We decided to not couple core to the outbox release. We both agreed to expose the settings via the settings dictionary. I talked to @johnsimons and @SimonCropp today and we came to the conclusion that it is better to have extension methods in the ravendb repo which then are obsoleted. Exposing the hack over the settings seems quirky as well. So this seems to be the lesser evil and more controllable.


No FIXMEs please.  If it really needs fixing, fix it.  Otherwise it is misleading.


@javathunderman: Why don't you simply use the 4-clause BSD license?


Ok


s/Fixes/Won'tfixes/


>we came to the conclusion that it is better to have extension methods in
the ravendb repo which then are obsoleted.

That was the missing piece, I'm good!

On Tue, Apr 21, 2015 at 2:34 PM, Daniel Marbach <notifications@github.com>
wrote:

> @andreasohlund <https://github.com/andreasohlund> I'll be your memory
> here ;) We decided to not couple core to the outbox release. We both agreed
> to expose the settings via the settings dictionary. I talked to
> @johnsimons <https://github.com/johnsimons> and @SimonCropp
> <https://github.com/SimonCropp> today and we came to the conclusion that
> it is better to have extension methods in the ravendb repo which then are
> obsoleted. Exposing the hack over the settings seems quirky as well. So
> this seems to be the lesser evil and more controllable.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Particular/NServiceBus.RavenDB/commit/da5b2a7decc5819b5c06965aa26e07b0d52a10ee#commitcomment-10823867>
> .
>


Cool, when did they add support for the caret operator?


Me gusta tu comentario.


Why do you add it? It doesn't seem to be used anywhere.


You **can** or you **cannot** assign `this _count`?


I believe these type of comments exist to placate a linter.


Changed in e339552ac7e1fc086ae7ebff4259db3762eb7cd6.


嗯我想了一下你说的有道理，查了下维基百科，和“色系”最近的是“色彩系”，对应英文词条是 Shades of color，这里可能确实不太合适，虽然似乎各种译文里色系不一定是邻近色，“配色方案”还是更稳妥一些


Haha,

why would I create a design for such adhoc scripts?


@yorikvanhavre

Noticed, in this file, that the 'corner' shape you modeled in FreeCAD used IfcFaceBasedSurfaceModel

...the same shope i modelled in Revit used IfcExtrudedAreaSolid

Could the answer lie some where in that difference?

![image](https://cloud.githubusercontent.com/assets/507113/7252272/0ce05b86-e7f9-11e4-84a4-0dd221228efe.png)


Ah, thanks for the spec link.  I'll make sure my Node.js adapter satisfies that then.


Y a mí! XD


Oh, this is fun. In `VectorList`,
```ooc
count: Int
_count ::= this count
```
so the property is the one with underscore. This is wrong, and most likely the source of this confusion. Someone was fixing this, but it never made it into upstream...


You are right, this leaked from some tests I was doing


Yes, we are trying to use whole words. You can use common short forms (i for counter, we also have idx and min sometimes). But I would prefer to keep names in their long form. We also use camel case for all variable names (So no _).


I deployed this update to staging last night. I'll try to get this onto production tonight.


1.2.0
Add Visual Studio solution


要不改成制图美学或者制图艺术博大浩瀚？色彩搭配方案似乎窄了。突然出个美学确实突兀，归根结底原文这句就突兀……

一般情况下还是忠于原文吧，如果原文写得实在烂，或者我们没时间没精力译到合理再自己发挥。


delete empty line?


g


@otizonaizit Can you explain what this line is doing?


There would be no memory problems with that. The whole menu, including the reference, is garbage collected once it is not needed any more.


I always assumed that we have the convention to name all non-cached cmake variables using lower case letters only. Am I wrong?


我对这段自己写没意见，反正是第三方工具的介绍比起正文更像附录。是不是初版是30种颜色？要么是单纯的不识数……


Det har varit såhär hela tiden, vet inte om det var önsketänkande när jag skapade tabellen haha!


FEO!!!! @xavilopez


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14058/testReport/) for this commit**


Shouldn't that cause some doc changes as well? Did you forget to commit those?


OOPS!!! product_product_model!! (Duplicated)


You missed the copyright and license in this file


Neimen så bra da, Per! :-p


Hoe krijg ik hier dan een speciaal pijltje? Bijvoorbeeld een afbeelding, of in ieder geval in het font (zoals in de design pfd). Doe ik dat met (afb) dan met background image bij de controlArrow?


Use print()


嗯按照之前的讨论就把“色系”换成“方案”？渐变色方案，分裂补色方案，定性色方案


I thought that was the intention of this method. Feel free to change it back and add change the comment on what the method does.


Why is this overload needed? It does not exist in the ```Pci_session``` interface.


The comment says to listen to `KeyCode::KEY_BACKSPACE`, but the example code shows `KeyCode::KEY_ESCAPE`. Confusing.


You probably mean `bindAddrs` instead of `bindAddrs_`?


Please remember to review your code before each commit, so that you can catch those mistakes early. I described in detail how to do that here: https://github.com/Mei152/Bloccit/commit/05b442208dc1ec68f0f72bcb4b7dee3a4a29153e#commitcomment-10640473


透明渐变这里问题比较多，ie下的显示效果不对，标准浏览器中的效果与设计稿的要求也不一致，建议重新关注下这块儿的实现。例如在标准浏览器中：
background: linear-gradient(to bottom, rgba(255,255,255,0.7), rgba(255,255,255,0.5) 75%, rgba(255,255,255,0));


Then the linter should be configured appropriately, or its output filtered, to avoid such pointlessness.


Have you considered to use the ```Xml_node::for_each_sub_node``` method for iterating?


表现不为表格的内容，最好不要用表格布局


Please back-port this commit into the maintenance branch.


Wrong spell of form and it should be configuration not configurations


th标签应该放在thead标签中，这个tr完全是多余的


The wording of this sentence confuses me a little: the compiler produces a set of tools?  I think you mean the LLILC project has the goal of producing such tools, primarily the compiler itself, right?


@bluejamesbond @lkolbly most of this is really good! Only thing I am worried about is the independent VAS declaration in sched_task (`task->vas_struct`). Since sched_task also incorporates a pointer to a pcb, which in turn has a vas struct already (`PCB->stored_vas`), this might cause problems. The way that we're doing it on pcb_exec is to [create a new VAS for the process during process creation.](https://github.com/rellermeyer/course_os/blob/pcb_exec/kernel/process.c#L299) The idea is that until execution the process will not need a VAS. Additionally if you look at the (defunct) [change I made in priorityQueue.c](https://github.com/rellermeyer/course_os/blob/pcb_exec/kernel/priorityQueue.c#L109) the intent was that dispatch deals strictly with processes that have been initialized with execute_process (i.e. stored_vas has been initialized).

How well do you think this fits with your current implementation of the scheduler? It was easier for us to keep the VAS as part of the PCB struct because of the various functions in process.c that use it, but if you have a reason for using a different VAS in sched_task let me know.


@MPParsley this was reverted. I've opened a ticket to figure out how to do this a different way.


do not track


Had some uber verbose logs there and removed them. Can discard that commit. (or force some logging)


Looking at `config.json.pgsql-template`, we don't have a `revision_db_name`. Correct me of I'm wrong, but shouldn't the DB for the revision tables and the actual alters be the same? I imagine only the schema would differ.


Wrong function description.


its a form function it must pass some parameter.


This should be gracefulReload instead of reload.


Project-foo is not a great example, and swap the order so the simple one is first.  Also, would remove the joke in the description.

Suggest something like "web-team-dev" or similar (something that has overlap with the use cases someone might try).


Blank line must not have any space character.


@chelmuth I'm fine with letting d3m go. Do you approve?


@lodoyun Hi! can you tell me what is visible result of this change? I am looking to update Licode often to gain performance/quality but I don't know if this commit worth it. Thanks!


Why is this reverted? Was there an issue with the code. or is having this configurable un-welcome?


Wrong commenting style and spell mistake


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14060/testReport/) for this commit**


assign options.stdlib[functionName] to a variable


Wrong commenting style.


Do we really need an option to toggle this? If we combine the row 'mark as watched' and 'mark as downloaded' it doesn't take up any extra space and as far as i'm concerned there wouldn't really be a need for an option to disable it


PHPCS: Expected 1 newline after opening brace; 2 found


PHPCS: Usage of ELSE IF is discouraged; use ELSEIF instead


PHPCS: Expected 1 newline at end of file; 0 found


Try to use single inverted comma in string.


Thanks buddy


85 lines x 80 columns tidy database. It contains the T1D incidence retrieved in 80 countries and the list of 77 environmental indicators used in the correlations of environmental factors and T1D incidence among children.

Used in the published study:
Diaz-Valencia PA, Bougnères P, Valleron AJ (2015). “Covariation of the Incidence of Type 1 Diabetes with Country Characteristics Available in Public Databases”, PLoS ONE, 10(2): e0118298. doi:10.1371/journal.pone.0118298.

Available at:
http://journals.plos.org/plosone/article?id=10.1371/journal.pone.0118298


Wrong commenting style


避免使用＊选择器，效率低！
这里将所有的box-sizing设置为border-box的目的是啥？不习惯标准盒模型么？非必需的情况下不建议这么使用。


Sí, así podemos implementar allauth tranquilamente ;)


Wrong function description.


I am not a native speaker of English, but I think it will be better to word it as "to fix the issues listed above" instead of "to fix issues" which is not very clear about which issues we are talking about.


it's quite unclear, so if options is undefined isVariadic is undefined ?


No license update for Johann C. Rocholl or Florent Xicluna?


个人喜好吧，其实我只是觉得定义时 `#1#2` 之间有空格，队形不统一而已。


I am not sure about it can u make it verify that every array element must ended with comma


parameters? anyways it's not really consistent


Awesome! I was just looking into how to fix that issue. Can I ask how you did it? Manually, or did you add some fix to the python code?

I'd love to get this back into upstream (in a reproducible fashion)


萌萌哒……

—
Sent from Mailbox

On Tue, Apr 21, 2015 at 9:02 PM, Qing Lee <notifications@github.com>
wrote:

> 个人喜好吧，其实我只是觉得定义时 `#1#2` 之间有空格，队形不统一而已。
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/CTeX-org/ctex-kit/commit/c6b1b1da7a731db0d32818c05e5804fd9f113433#commitcomment-10824278


This was changed in a later commit
On Mon, Apr 20, 2015 at 11:58 PM Georgios Kalpakas <notifications@github.com>
wrote:

> Why not set selectedChip to -1 ? Is there a special meaning to null ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/angular/material/commit/c62d4dfd0f8741d90a2c69873154f55c91c3e7e7#commitcomment-10818070>
> .
>


Wrong function description


@djowett we're gonna deal with this in a different way


Can't we use meaningful variable name.


@djowett feel free to update this on master branch


Typo, "canola" instead of canols :P


I was also thinking that maybe the case of ```libusb_kernel_driver_active``` or ```libusb_detach_kernel_driver``` returning ```LIBUSB_ERROR_NOT_SUPPORTED``` needs to be handled gracefully (ignore the error and return 0) in order not to cause problems on system other than linux.
Can you verify if this is actually a problem on non-linux systems?


如果考虑到旧版本浏览器，建议加上前缀


if the master is stable(?), maybe you could push a new release including this change to `maven` repo? thanks!


Can't we find out $filter_key_array size first in variable and then use that variable in for loop.


这里的负外边距有必要吗？


parameters


id= is undocumented, right? I don't see it in the documentation.

Generally, combining these conditions is unintuitive for me. If you combine appMatches with && you have to change its definition in case of selectors == None, of course.


I like this documentation :) I do not like the naming of "Id", though. It is just any value, right? Could be a version, anything.

What about escaping?


we can use empty function here


Not yet, did not know that it exists at all.


you are calling this "Set" above. The terms should match. (I would even try to match the case)

What about allowing String literals for values which are a bit more involved?


ignore anything not supported; add cb7 output


Everything looks OK to me.


标题用语义化的标签


Testing commit


Hello @Drugoy ,

Thank you for your answers. Currently I'm experimenting with a different approach, but I think I will take this way at some point (sooner or later)


This nil check suggests that `actionChan` gets closed somewhere.  But I can't find a relevant `close`.


It was used during development for not changing the clients. But now it can be dropped and all device drivers have to be adjusted accordingly.


多余的div


k


Je dirais plutôt ré-apparaitre


Мы работаем с длинными числами, функция не будет работать.


this is not originally my code - it was moved from original file to this else branch


description: "an empty program",


I should probably comment on the above.  In python 2 calling sorted() with a list of dictionaries would sort in some general manner and not cause a problem - but in python 3 it will not allow you to sort a list of dictionaries because dict() and dict() are not comparable types.  recur_convert will traverse a data structure and convert any dictionaries (nested or not) to a list of tuples.


This commit doesn't match its description.


s/Won'tfixes/Worksforme/


hahah epic chane rite here dis one


Yeah! Thanks.


It's not expected to happen in a running Weave, but the unit tests call Stop() which sends a nil.


Makes sense. I used Bloc's method. The course material doesn't state this
(I don't think), but you need to write the following code in the vote test
for it to work.

before do
       post = Post.new
       @invalid_vote = post.votes.build(value: 21)
 end

On Tue, Apr 21, 2015 at 8:31 AM, John Sawers <notifications@github.com>
wrote:

> I think that's more than you need. If you create one upvote and then check
> the value of post.votes.first, then do the same for a downvote you'll have
> all the bases covered.
>
> > On Apr 21, 2015, at 08:24, Piper Gragg <notifications@github.com> wrote:
> >
> > If I were to do it my way, I'm guessing I'd have to iterate through the
> votes that belong to @post and check each of them to make sure they have a
> value of 1 or -1. I'd probably want to abstract that functionality out into
> a test_helper.rb file and include that back in.
> >
> > Let me know if that sounds right!
>
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pgragg/bloccit/commit/688c31ee8d7e1752aba328b26c5fbf94319213d2#commitcomment-10823835>
> .
>


The comments were originally there for the VM Unit tests so that one didn't have to analyze assembler program just to know how it works. I agree there is no more room for this in integration tests. I will remove that.


What about Gatling DSL usage outside Simulations (traits, objects)? We don't want to require Gatling users to use something like a cake pattern...


Hi Priyanka,
Please update also the test for `HkConverter`.
```
Starting on 1 Nodes at : 2015-04-21 15:08:28.978323
Traceback (most recent call last):
  File "/afs/physnet.uni-hamburg.de/users/th1_li/ikrivenk/TRIQS_build/dft_tools.git/test/hk_convert.py", line 27, in <module>
    Converter = HkConverter(hk_filename='hk_convert_hamiltonian.hk',hdf_filename='hk_convert.output.h5')
TypeError: __init__() got an unexpected keyword argument 'hk_filename'
```


These numbers for JRuby are almost certainly way off. JRuby 9k.pre1 did not have any focus on performance, and the JIT has a number of known performance problems. Those have almost all been fixed for pre2, so I'd love to help re-run this bench.


Looks goor


这里直接给a标签设置background，可减少一个img标签；
同时这里a标签内不要留空文字，给文字加上取值足够大的text-indent，保证样式未加载时，链接依然可用


What happens to the old name fields?


*good


Выдает ошибку. Я тоже прочел, что инициализировать `final` переменную можно позднее, а не при объявлении, но потом переприсваивать уже нельзя.


yes it's MIDAS file :)

to make fasl
```
:midas
dsk:msendi lisp foo;
```
and
```
(load '((dsk foo) msendi lisp))
```
etc


I thought it wouldn't work right if it wasn't == instead of === because our keys are in strings and our obj.m is an int.
{m:200}
{m:200,"200": 1}

Now with === those paths will return 0 in compare paths. With == it will enter compare paths and the first object will be less than the other as intended.


Will be removed,


TeamCity xBIM :: XbimExchange :: Build [Build 246](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1465&buildTypeId=XBIM_XbimExchange_Build) is now running


Not following your logic here mate, when would pm2_config be defined before hand?

The last @pm2_config is redundant due to the implicit return of the assignment.


Why is this `select` needed?  Given that it contains only a single `case` clause, can't you simply say `action := <-actionChan` without it (or even `(<-actionChan)()` if the nil check is superfluous).


I think what you have added is great, I am in agreement with your comments.


After some investigation, I believe https://github.com/abesto/yed-aws-palettes/commit/35a41f11576784a396cfbb0fe7c8ca113098270a fixes the issue. Please let me know something's still amiss :)


Review done


Shouldn't we rename database to DB? Or will the abbreviation to close to Deutsche Bahn?


TeamCity xBIM :: XbimExchange :: Build [Build 246](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1465&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:15


Yes, I think that an extension for `yast-rake` would be a good idea (`yast-rake-ci` ?), it would define the rake task for running the checks (`rake check:ci`) and define the package dependencies.


+ Performance Anstrebung durch SETRANGE anstelle von SETFILTER


要不咱们也别用 `\hypersetup` 这个名字了，换成自己的名字吧。


No....But you need to add the fix for it. It is somewhere around...lol :)


@gc00: I don't understand this change. In the current HEAD, when I build using --enable-m32, I do not get any files in the bin/ directory, so where do we get the symlinks from?


The values should be constants (because they are constant).


Work in progress*

Purpose:
Banking system designed with socket connections

How to use:
1) Download both jar files, banking_client & banking_server
2) launch banking server
3) launch banking client
4) create or use existing account
5) deposit or withdraw $$

To be done:
Server writes data to files, serialization.


I fear this will break everything.


yeah during the manual split it was not included.


Done.


I think all of these emitter instances should be constants. Maybe the emitter class manages the instances?


Not a problem,  there's always room form improvement. With the re-factor pull request all of this is really for not though. But it seems to work for me on Ubuntu 15.04(unpatched) without issue. I'm going to test the re-factor code and see it results in lockups.


@jpfr still a problem with the client - the identity must come from an endpoint


Added editor-style.css task


Interesting idea. But wouldn't it be more natural to wrap the Pci::Connection in a new class-local struct?
```
struct Platform : Pci::Connection
{
  Platform()
  {
     ... do session upgrade here ...
  }
} platform;


debug CLI switch was broken by this change


Ewwwww.


Me parece re bien que sea un modulo lo de Multimethod pero me parece que habria que agregarlo a Class en vez de a caada clase en particular cosa que cualquier clase tenga la capacidad de usar multimethods.
En los ejemplos del TP no le agrega nada:
```ruby
class A
  partial_def :concat, [String, Integer] do |s1,n|
    s1 * n
  end

  partial_def :concat, [Object, Object] do |o1, o2|
    "Objetos concatenados"
  end
end
```
O en todo caso ver de agregarlo de forma dinámica el modulo a cada clase que use multimethod pero me parece qeu es mas complicado interceptar cuando clase no sepa como usar partial_def y ahi agregarlo.


I just borrowed it from https://github.com/dotnet/llilc/blob/master/README.md. Agree that it is confusing though. Any suggestions @russellhadley?


I think our requests will lead to a conflict...


Like Emitters::AmPm.lowercase and Emitters::AmPm.uppercase, which are factory methods that returns the emitter (could be a lazy-initialized instance that the class caches, or a constant).


Guardando l'esempio della risoluzione del sistema non lineare ho notato la possibilità di definire un tipo legato ad una funzione. Ripensando al puntatore che ho creato nella prima versione ho cercato di migliorarla definendo questo nuovo tipo funType.


beautiful


@lholmquist +1


golint can't be configured, unfortunately.


It's more about dependency injection than it is about cake pattern.
I suggest providing the users with a clean entry point (from what I saw, the simulation's constructor looks like a good candidate but there may be better ones, I am not familiar enough with the code base to tell) and leave the responsibility of wiring their dependencies to them either through constructor injection, cake pattern or using their own mutable state on an object as they prefer.
A possible transition path would be to keep the  `Predef.configuration` but make it deprecated and link to a document explaining how to not use it.


This seems to be a comment, not sure though


Thanks for this, especially for adding tests to it (and correct my typos there)


Thanks for catching the typo. This is fixed in pull request https://github.com/cocos2d/cocos2d-x/pull/11539


What do you think about adding a utility function for determining the resource ID for a given PCI bar to the Pci::Device class? This would eliminate several code repetitions.


Can  you add a few more lines,  to explain what you did?


@byorgey I think this should fix our issue with the code sample.
Please give it a try and push it up to the web.


nice!


[Notes](http://stackoverflow.com/questions/185934/how-do-i-create-a-copy-of-an-object-in-php)

Good one.

In PHP 5+ objects are passed by reference. In PHP 4 they are passed by value (that's why it had runtime pass by reference, which became deprecated).

You can use the 'clone' operator in PHP5 to copy objects:
````
$objectB = clone $objectA;
````
Also, it's just objects that are passed by reference, not everything


@anton-bondarev Please setup your IDE to fix whitespace errors and amend this commit!


I'd much prefer that this be go in alloc.go.  Also, can't it be a `close(alloc.actionChan)`?


Could we use a reference for ```pci_drv```?


Nice!


Same as above.


Thanks @marcoow !


Instead of sharing the field "reportDir", prepareReport() could return the directory which would be propagated to a parameter of persistFileSources().
My 2 cents.


I doubt that the scenario from issue #1

    <title> Hello {{if title then {{ | }}}}</title>

    { }

actually works for you instead of throwing an error like it does for me. But you consider that normal behavior, hence the Won'tfix.


@Zolisa001 remove log


It returns the object var unless its defined so if it ever gets called more than once it just returns the instance var instead of "calculating it" again.  In this case you might be right it may never actually get called more than once but it seems to be a common practice in hwrps.  I can clean that bit up to always just return the string, I just wanted it consolidated so its easier to manipulate later.

I'll have it just return the string every time, in this instance the extra code isn't very helpful.


Maybe I'm too bored at work... :sleeping: :


Then filter its output? (Or ditch it for its silly policies.)


Basically, `Predef.configuration` is not meant to be used directly by anyone.
It's set right after Gatling's startup and is only used to provide the configuration to the DSL parts that need to access it.


Gatling is not intended for the sole seasoned Scala developers. And we can't afford such a breaking change a you suggest for all our users. We don't want to raise the required technical skills any higher.
So far, the only change in public APIs in 2.2 is fixing a typo (`connection` to `connectionHeader`).


@Zolisa001 you can leave this blank as the default `<h2 id="heading"></h2>`


They are still there ;-) With this change **all** 'name' labels will change the value. For replicationController it means the label not longer match with Selector (and the osc create will fail). I switched to use 'template' to avoid conflicts with existing labels.


Please could you send a PR for this fix?


I left an explanation in that thread: https://code.google.com/p/android/issues/detail?id=75000#c5

Basically, we were using seekg() incorrectly, and lucked out on desktop because std::ios::beg happened to be 0.


How about changing:

> Help us improve our repository and contribute new styles or fix up existing ones. We're particularly interested in styles for journals and published style guides and may refuse some contributions.

to

> We welcome submissions for new styles and corrections to existing ones, and are particularly interested in styles for journals and published style guides. If you wish to submit a style that doesn't fall within these categories, please first check out our [Criteria for Inclusion].

I also think it would be good if https://github.com/citation-style-language/styles/wiki/Criteria-for-Inclusion mentions the types of styles that we do accept. It should also mention that we accept university and department-specific styles, since we get a fair number of those.


Thread local storage is currently only supported using the standard pthread_ API. I haven't yet investigated the compiler specific `__thread` extension (http://clang.llvm.org/docs/AttributeReference.html#thread), but I'll put it on my TODO list, it is a nice feature. I don't think there should be anything that would block from us being able to support it.


````
KEY `supplier_id` (`supplier_id`),
````
should be:
````
KEY `supplier_id` (`supplier_id`,`supp_reference`),
````


"Claim" seems like a tricky case.  I am surprised it is not discussed further in the doc.


Test comment


It got overrided.Okay thank you.


TODO: "not implemented"


hmm is this a good default?


Agg, sorry man. Okay.


I did think of that at first, but then I would have ended up with a `Translation` object that, AFAICS, I can`t really render in a clean way to it's plaintext version.

In the end, I stuck with this version because I did not want to issue 2 calls to the localization service for the same key.


You assert that these are "effectively the same", and I wonder why. Did you use any standards? I'd appreciate links for more documentation on these changes.


The new link worked for me :)


What's the general schema? Instruction returns next node or it is provided earlier?


With `git-link-use-commit` baked into `git-link` is this needed?

I can see one saying `git-link-use-commit` is global, and one might not want this behavior globally (not sure if I agree with that), but to that I would say why not make it buffer local?


This should have actually been in a different branch as it is not related to the testing issue


UP


Thats what you get when you don't check over your work - "Sonic the Fasthog"


Thanks!


![tmp preview](https://cloud.githubusercontent.com/assets/369527/7253352/f9b0432e-e833-11e4-8a56-501a2116df2e.png)


That's sample output of the work above


looks good


Hyggelig av deg å forlate filen i bedre stand enn når du åpnet den; anbefaler å gjøre slike ting i egne commits tho så hvis du for whatever reason må reverte GTM featuren slipper du å reverte cleanupen :)


Note that this might break compatibility (or show warnings) with any Django version before 1.6 and the readme states that django-guardian should work with >Django 1.2


Не могу сделать защиту от дурака.
Если выбирать все правильно, то прога работает корректно.
А как сделать, чтобы нельзя было выбрать не правильно (например, если юзер не выбирает чем блокировать, а сразу жмет на кнопку "Блокировать") пока не пойму как.


Hey Alex, a library I'm using has been using goraft, but its been really unreliable(especially with reconnections and leader selection) because its unmaintained now.

I see that you're the most active branch of this project, are your fixes oriented towards leader selection, or could you let me know what kind of improvements you're making to goraft?


Explain this.


Yeah, pretty much. Every 1st level entry (in this range) is pointing to the same coarse page table. The "| 1" sets the lower two bits to 01, which is what says hey, that's a coarse page table.


Думал из-за того, что использую одновременно wchsr_t и wstring, где-то читал, что это не рекомендуется, убрал стринги - результат отрицательный.


call this `startThentosSessionByUserId`?


I did this and then realised @kpwebb had just done the same thing upstream ...oops! Will eventually delete this whole fork.


@indutny It's my pleasure. Thanks for your feedback.


whitespace.


I guess so... Say you can add the same title for all the pages via code instead of specifying it in the shortcode. Or overwrite it for something specific programatically?


if the master is stable(?), maybe you could push a new release to the maven that would include this change ? thanks!


Somebody should remove the priorityQueue.c file and post on Piazza to make sure nobody in a different branch will try to use it.


@petschki I was revisiting this, but I'm not sure if I fully understand it. Is it possible, that it covers the same case like this method here? https://github.com/plone/plone.app.event/blob/c29ce46aec1d1ccd5792fbf2d1ee3106a77e16c0/plone/app/event/base.py#L404
``start_end_query`` does not support ``minmax`` or ``min:max`` queries, though. What do they do anyways in this context?


Hm, sorry I don't understand. Could you explain with a bit more details :)


`getFromSession >>= decode`

(you once said that mixing `do` and `>>=` is bad, but i think 4 extra lines of code are worse...)


Not sure I like this simple print approach. Isn't raising a warning better?


Yes, but I doubt that

```js
> if (title) { console.log(title); }
ReferenceError: title is not defined
```

works for you either, so it's more of a MatchesStandard thing, a non-bug.


This looks straight forward enough


Does that actually type-check? My inner type checker might be wonky today.


if we want to still test this, we can always fall back to plain old RS. Instantiating the FuturePublisher directly and wiring up with the probe. I am not sure we want to do it, just noted.


This will set the wrong attribute value in case the default db column value is bigger than 0.


Indeed !


LGTM.


fixes #25


Okay. Well I wasn't sure, and it wasn't working at all before the change.


Зачем? Обфускация не дает безопасности, намного меньше страница не стала, а без этого удобнее визуально просмотреть страницу.


a simple equality check since it is an object, and you don't do anything in the else case?

    if (realPublisher ne RejectAdditionalSubscribers) realPublisher = ...


LGTM


fork and create a pull request


nit: name NewServerWithChannel.  Also, doesn't need to return an error


ah.
you may be right.  perhaps this:

    (>>= decode) <$> getFromSession

I think `getFromSession` types `m (Maybe a)` for whatever `m`.


início


It's a bug in your templating engine, because I think templating engines should be more flexible than standards.

With macro errors, it's a better experience if you drop the macro but continue to template the rest of the file, e.g. with:

    <!doctype html><title>Hello {{if name then {{= name in html}}}} !</title><rest-of-page/>

    { }

it's a better experience to log the error, but send:

    <!doctype html><title>Hello !</title><rest-of-page/>

The current experience is logging the error, and sending a blank page to the client, so a tiny mistake in a complex and potentially huge templated page can cause everything to break.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14064/testReport/) for this commit**


Hi @stephencelis .. So I changed a few things:

* Changed the deployment target from Automatic to 8.0 - Because I am unable to change this value on dependencies that are pulled in (automatically on a CI server)
* Set `APPLICATION_EXTENSION_API_ONLY` to `YES` to make the code compatible with app extensions
* Removed the SQLCipher targets

The first two may be things that you want to cherry pick from my fork. It makes the project more compatible. I don't think there is anything 8.0+ specific in the project anyway.

Disabling SQLCipher was more a workaround for us. The project without SQLCipher builds fine locally both for simulator and device but when we do CI builds (on Xcode Server) the build always fails on SQLCipher. I tried to do this the right way (not sure what that is though) without success so I ended up just removing the targets completely.


I didn't quite understand that.

With this extra function, the original behavior of `git-link` is retained. I always end up using `git-link-force-hash`.

I prefer not to change the global value of `git-link-use-commit` in the event I need to use the original `git-link` function behavior.


good catch


Or I can simply move the `git-link-use-commit` definition to my personal init.


@sprakes You bring up a fair point. Waiting to create the new vas until execution is actually a really good point. In the event, someone calls `prcs_exec` (`exec` in Linux) the it is smart to delay the vas creation because we don't need to copy over the parent process state at the time of the function call. However, this may not be the case fork `prcs_fork` as well I feel that we will also need to duplicate the vas (or something similar) to store the exact state of the parent process (which would be at the time of creation). We will still need to discuss forking with Lane but I presume we will need to copy the vas at the time of the function call. So, I think the solution would be a mix of what you said and what is already in the scheduler. Additionally, I pull your changes in `process.c` tonight and move some of the process-specific code from scheduler into it. I don't forsee any issues with the refractor, but I will keep you posted.


nit: rename to NewTChannelOutboundProtocolWithChannel


fix looks good!


@lukasspiess How about zipping the data before it gets persisted in order to also minimize the file size. The persistence layer seems to be quite busy when the app tracks a lot of data items.


Is there a better way to do this?

Every time I run a rails command I get this message, which is unnecessary.

Its also making a few of our specs fail:

```
Failure/Error: r.out.should == ""
       expected: ""
            got: "Could not load redis ruby gem. Some features are disabled.\n" (using ==)
       Diff:
       @@ -1 +1,2 @@
       +Could not load redis ruby gem. Some features are disabled.
```


this is shit


if we keep this version of the function, we'll need to have a way to specify that the channel should not open a listener.  We definitely don't want to listen on all interfaces.


AAAARRRHGHG


@slandelle your arguments in favor of not making it harder for less scala-savvy developers make a lot of sense, I hadn't considered that.

In 2.1.5 I was unable to find `Predef.configuration` and had to go through `GatlingConfiguration.configuration`.
In 2.2 the latter doesn't exist anymore and seems to be replaced by the former. @pdalpra Both of these class are public, the configuration member of both is public too. Arguing that `Predef.configuration` is not part of the public API when it is imported automatically by all the samples is a bit surprising.

I realize my comments especially the first one was probably too strong. Would you guys be open to a PR that allows a single argument constructor on simulations allowing to pass in the configuration while preserving the existing behavior ? note that this would be one step further to allowing multiple gatling instances from running in parallel (I guess at the moment the instances would overwrite each other's configuration during the run) this would be useful for #1474.

I think using the config system for both user code _and_ gatling configuration makes a lot of sense. It provides the user with the benefits of config : cascading config, including automatically and cleanly fetching values from parameters or environment variables and a coherent and logical API to retrieve values from the config.
I find this much better than mucking around with System.getProperties  or Integer.getInteger or java.lang.Long.getLong etc etc to read run specific configuration data. (NbUsers, rampTime, baseUrl, etc.)


LGTM.


general


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14065/testReport/) for this commit**


Gracias


Even easier - in Python [] is falsy, so
```if temp_content:```
should suffice


alternately, have a single version of this function that takes an Options struct which has members for the Channel (nil means create a new channel) and the timeout (which needs to be optional, as per comment below).


There is no point editing files in the `_site` directory - The contents of this directory is generated automatically by Jekyll (every time we commit something to the `gh-pages` branch).


Because it's more flexible to allow (re)binding buttons with bind[].  Nearly ready to merge the feature.


This timeout needs to be optional.  In the case where a server is making a call to another system, you don't want a per-call timeout - the context passed to the Handler has an overall deadline for the request, so you want to use that context unmodified in all outbound calls from that Handler so that those calls inherit the overall request deadline.


:) Cool


Hmm, any other spots we should include `sanitize_text_field`?


also, this func shouldn't take a bindAddr


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14066/testReport/) for this commit**


We might describe each one briefly, but I would be careful about comparisons which could be misconstrued as criticism, especially since we're going to be probably relying heavily on one of them (docker-compose).


readme


Remove ELLIPSIS and add them into doctest-TC


I prefer this one:
The Google APIs client library for .NET provides simple, flexible, and powerful access to Google APIs such as Drive, YouTube, Calendar, Storage and Analytics.

what do YOU prefer?

+ I don't think that we need the "A link to the list..." sentence, cause we have the links later on this page.


"or" between them?


right, but FuturePublisher is supposed to be rewritten anyway, and then it should not bother about multiple subscribers


please refactor: uris should be specified on command line


This function does not return a "parser" nor does it parse anything.
More like ``open_sheet``...


Actualización 1 realizada


I guessed so.


You miss a logic method here, like ```$job->isOwnedBy($user);```


this should still be staff required, right?


Gotcha, thanks jf


行啊，这样 `hyperref` 换了内部实现也不怕。


> Cool, when did they add support for the caret operator?

A while ago, not so long :)


See line 27 below. Do we need to include the file "sftpdlg.h" twice?


TeamCity ChameleonForms :: Continuous Integration [Build 242](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3576&buildTypeId=bt20) is now running


I think better use VS2013 Community Edition https://www.visualstudio.com/en-us/products/free-developer-offers-vs.aspx


Have you tested  that the lucene-indexing still works with prefixes for the metadata files?


```.haskell
(flip toLEdge) () == flip toLEdge ()
                  == (`toLEdge` ())
```


Should the two occurrences of `4` on this line be `5`?


I changed the word the C code uses to be intcap_t which results in the capabilities being copied by clc/csc instructions.  I verified that the compiler emitted the correct instructions when I used this code to implement memmove.  That being said, we could really use some c99 str*/mem* tests.


GitExt should suggest do that automatically when you clone repository


We reached the decision that professors aren't staff, didn't we?


Can we kill this pull request now that you created a new one?


This might be better with `guard` rather than pattern matching on `True`.


You're right. Thanks.


This doesn't compile for us anymore.  Says it can't convert (bufferPtr + total) from IntPtr to byte[] and that the method-signature is wrong because of this.

Did I miss something in the commit, or is there something I should have done to get Vorbis stuff up to date?


Assume I'm wrong


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14067/testReport/) for this commit**


This line can be removed as you're defining `swap` down below.


Tests? :)


Thanks, this will be included in 1.4.


Yes, I have warned you about that.


I am going to delete this branch, do you want this?


opa, q legal! outros caras tipo o uwsgi não davam isso pronto... mas se ele já tem um -d pra isso, acho q fica fácil. Só ver se tem como parar e reiniciar facilmente... se não, aí tem q criar os scripts. Eu nunca lembro os nomes direito, mas transformar em "serviço" quer dizer usar o systemctl, systemd, init.d, essas paradas aí q eu não manjo direito, rs.


@flibitijibibo This doesn't compile for us anymore. Says it can't convert (bufferPtr + total) from IntPtr to byte[] and that the method-signature is wrong because of this.

Did I miss something in the commit, or is there something I should have done to get Vorbis stuff up to date?


noooooo


Does this use of timestamps mean that we are requiring that peers have their clocks reasonably in sync?  This is the first time I'm aware of that weave requires that.  Is it a safe assumption? Could we at least try to detect the case where clocks are grossly out of sync, and warn on it?


This method is unused.


I found another thing. Will add the tests when it is fixed :)


Thanks @niemyjski and @akoeplinger!


please refactor: pass URIs via command line args


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14068/testReport/) for this commit**


@sprakes You bring up a fair point. Waiting to create the new vas until execution is actually a really good point. In the event, someone calls `prcs_exec` (`exec` in Linux) the it is smart to delay the vas creation because we don't need to copy over the parent process state at the time of the function call. However, this may not be the case fork `prcs_fork` as well I feel that we will also need to duplicate the vas (or something similar) to store the exact state of the parent process (which would be at the time of creation). We will still need to discuss forking with Lane but I presume we will need to copy the vas at the time of the function call. So, I think the solution would be a mix of what you said and what is already in the scheduler. ~~Additionally, I pull your changes in `process.c` tonight and move some of the process-specific code from scheduler into it. I don't forsee any issues with the refractor, but I will keep you posted.~~

For now, I feel like it would be best to keep the vm related code found in process.c to scheduler.c since the scheduler will have to deal with timer_interrupt handling and stuff just before executing a process. The scheduler also deals with message handling which requires access to vm-related functionality. As a result, I think it would be best to move vas creation to scheduler but of course delay the creation of the vas until dispatch aka. sched_timer_interrupt_handler


# Querying Open Street Map with XQuery
## Welcome to Project

The aim of this project is to develop a library for querying OSM maps with XQuery. With this aim, we will define a XQuery library that enables to query XML data from OSM. This library is based on the well-known spatial operators defined by Clementini, provinding a ser of XQuery functions that encapsulates the search on the XML document representing a layer of OSM, and makes easy the definition of queries on top of OSM layers. In essence, the library provides a repertory of Urban Operators for points and lines which, in combination with Higher Order facilities of XQuery, makes easy the composition of queries and the definition of keyword based search geo-localized queries. OSM data are indexed by an R-tree structure, in which points and lines are enclosed by Minimum Bounding Rectangles (MBRs), in order to get fast answer time.

![Alt text](http://indalog.ual.es/osm/Querying_Open_Street_Map_with_XQuery/Welcome_files/shapeimage_2.png)

TEAM:

* JesÃºs M. Almendros-JimÃ©nez
* Antonio Becerra-TerÃ³n

* Universidad de AlmerÃa
* Dpto. InformÃ¡tica
* Crta. Sacramento S/N
* 04120 AlmerÃa

CONTACT:

* [jalmen@ual.es](mailto:jalmen@ual.es)
* [abecerra@ual.es](mailto:abecerra@ual.es)

## Library Download
### rTree Indexing OSM Layers
* [rTree Indexing Library](http://indalog.ual.es/osm/Querying_Open_Street_Map_with_XQuery/Download_files/rtree_library.xq)
* [rTree Indexing Generating Test](http://indalog.ual.es/osm/Querying_Open_Street_Map_with_XQuery/Download_files/rtree_test.xq)

### OSM Layers as BaseX Database
* [OSM XQuery Library](http://indalog.ual.es/osm/Querying_Open_Street_Map_with_XQuery/Download_files/osmXQueryLibrary.xqy)
* [OSM2GML Transformation Library](http://indalog.ual.es/osm/Querying_Open_Street_Map_with_XQuery/Download_files/osm2GmlLibrary.xqy)
* [XQueryTestResults](http://indalog.ual.es/osm/Querying_Open_Street_Map_with_XQuery/Download_files/XQueryTestResultsDef.xq.txt)

### Additional Tools
* [JOSM (Java Open Street Map)](https://josm.openstreetmap.de/)
* [BaseX](http://basex.org/)

## R-Tree Indexing of OSM Layers
Different figures representing the r-tree index on OSM Maps by using the concept of MBR (Minimum Bounding Rectangles)

![Alt text](http://indalog.ual.es/osm/Querying_Open_Street_Map_with_XQuery/Indexing_files/Media/FigureIndex1/FigureIndex1.jpg)

![Alt text](http://indalog.ual.es/osm/Querying_Open_Street_Map_with_XQuery/Indexing_files/Media/FigureIndex2/FigureIndex2.jpg)


I prefer this description:
The Google APIs client library for .NET provides simple, flexible, and powerful access to Google APIs such as Drive, YouTube, Calendar, Storage and Analytics.

* you can also delete the "A link to the  list of ..." because it's mentioned below.

What do you think?


should we have an "or" between them?


TeamCity xBIM :: XbimExchange :: Build [Build 247](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1466&buildTypeId=XBIM_XbimExchange_Build) is now running


Okay - would still help to commit the tests and the bugfix always together. That makes it way easier to comprehend what the issue was. Especially when we look back at old commits in the future :-)


I have changed UserProfile as the main table, and modified the methods according to the change, can you please check if this is okay then i can merge this to master


[supported APIs] ==> [All supported APIs]


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14069/testReport/) for this commit**


google-api-dotnet-client Announcements  ==> Official announcements  ?


Aw crap. Lol
On Apr 21, 2015 10:06 AM, "Stephen Sawchuk" <notifications@github.com>
wrote:

> https://docs.npmjs.com/files/package.json#name
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/callmehiphop/workhorse/commit/e873ebe38adc994f81c5c83c8851f986b128bfa9#commitcomment-10826019>
> .
>


Indent on new line. #Pylintpolice.


nit: return


TeamCity xBIM :: XbimExchange :: Build [Build 247](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1466&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:29


`log.info` seems more appropriate in this case, but this is already nested within a "verbosity" keyword, so I'm OK with the print statement


The calculation to check whether it's aligned appears to be using word, so it may not DTRT for word-but-not-capability-aligned copies.  I've now updated the assembly one.  We have a fairly extensive test for memcpy in the test suite, it might be worth copying it into cheritest.


cmake_find_energyplus_or_EnergyPlus (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: 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)


Makes sense. Updated.


I'll just change it to:
A full list of all supported packages is available at [Google.APIs nuget](https://www.nuget.org/packages?q=google.apis)


태종형님처리


Ahh, the one bellow is redundant (it actually doesn't work for Windows). Will remove it.
Thanks!


тестовый коммент


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14070/testReport/) for this commit**


I'm sorry but the Throwable.addSuppressed() was introduced in Java 7. We currently compile against Java 5 and we can't raise it higher than Java 6. Please implement this feature via reflection and enable it only if this method does exist.


It'd be better to just write the rows to the response stream somehow.


This looks fishy. Synchronized won't protect you if someone later modifies the hash map via this reference. You shouldn't be allowing access to class internals at all.


If you find an issue with in the client library we ==> If you find any issue with this client library we

an issue here ==> an issue [here](https://github.com/google/google-api-dotnet-client/issues/new)


Is this branch necessary? I'd assume that `if (mName != val)` then isn't the also guaranteed to have changed?


Ahhh. So what we need to do is make sure both variables are of the same type before comparing them.

`(Number(minKeyA) === Number(minKeyB))`

Sorry, this is kind of specialized JS stuff here: in JS, most people feel that the type coercion that `==` and `!=` do can create unexpected and unintended side-effects, so most linters require using `===` and `!==` instead. That way there's no ambiguity when you're reading through code as to what the author intended.

Thanks for spotting this!


we ask you to post them to... ==> feel free to post them in...


why ist that relation an N:N ?


haha, maybe the author of this document could take a chill pill and relax ;)


This looks suspicious. Everything in this class is `synchronized`, so I assume it's supposed to be used from multiple threads, but you're allowing clients to modify this internal hash map through reference.


@franga2000 : Will be reverted soon , sorry guy T.T


Hope that my comments make sense.
Thanks for doing that!


You are absolutely right.


Why some of the files are completely red on github whereas only one or two lines have changed???


this is really strange that this is disabled... maybe it's better to do this in a separate bug because it will likely be a blocker


Yeah, just forgot about that import :-/


еще один тестовый коммент


Indeed!  Will fix that now.


Y'know, I think this is exactly what I'd have written, modulo the name of the `getTemplate` function.


fba00d3cd071130ba9bc8bed2f1eab6c04c28e08 改成内部名 `\ctex_hypersetup:n` 了。


Another option is that in `git-link`, the `remote` is set using `git-link-read-remote` if `(car current-prefix-arg)` is `4` (`C-u`). And the `git-link-use-commit` is set if that `car` is `16` (`C-u C-u`).


Also, the following line seems to be a bug and is what was causing volume: 0 config setting not to work for me.

https://github.com/frankyghost/projekktor/blob/ver/1.4.00/src/plugins/projekktor.controlbar.js#L1299

In that logical OR, the this.getConfig('volume') will always be falsey if you pass 0 in as a config value, as in the following.

projekktor('#player_a', {
        volume: 0
}, function(player) {});

So 0 will never be set as the value. It was either set the value from the cookie or set it to 0.5


It seems to be the file encoding which is different on Linux and Windows...


Why can't it be set to the default value?


as discussed with Bevis, I now think it's better to use `cdmaCategory === null` because we don't really know if `id` can be 0 for a good reason.

So I'd suggest something like:
```javascript
var isGSM = cdmaCategory === null;
var isGSMCmas = isGSM && (id >= 4370 && id < 4400);
var isCDMACmas = !isGSM && (cdmaCategory >= 0x1000 && cdmaCategory <= 0x10FF);
```


Ok, it was at least a bit faster. That's enough for now. The task details instead... it'll be messy to optimize it. We just have to live in the hope that nobody actually has or adds thousands of subtasks, which is a new feature anyway.

BTW, tried to find and add you to my contacts in linkedin. Found Dan. Psycho might be the one living in Bern. And peterdd has been very good at hiding himself, although I suspect that he owns or has owned a VW Kleinbus.


Your commit message got a bit long here. :)

And this might be a good example of when you want to use `--author` when you commit.


That almost works (it's `(>>= Aeson.decode . cs) <$> getFromSession "sessionData"`, but I'm not sure it's an improvement. I'd rather spend 3 extra lines and and save the reader some parsing and type-inference work.


Yes, although the timestamps are only used for tombstones - and the timeout for tombstones is sufficiently larger.

A note needs to be added to the docs along the lines of: "Host clocks need to be reasonably within sync; if the host clocks differ by more than 2 weeks, peer deletion will behave inconsistently.  Similarly, you should not allow network partitions to persist for longer than 2 weeks, as you may see previously deleted hosts reappearing."


it seems you've replaced tabs with spaces

On 04/21/2015 04:15 PM, Christopher wrote:
> It seems to be the file encoding which is different on Linux and Windows...
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/hevs-isi/gdx2d/commit/5168338c5e50f4d686f48efd5402358214bc79fd#commitcomment-10826206>.
>


Les screwdrivers n'y changeront rien, regarde l'explication sur le forum de VanessaE


Oops.  I meant to write "...defined a matrix from an image".  Shit.


same thing here: we should only distinguish with cdmaCategory:

```javascript
var isGSM = cdmaCategory === null;

if (isGSM) {
  return GSM_PRESIDENTIAL_ALERTS.indexOf(messageId) !== -1;
} else {
  return CDMA_PRESIDENTIAL_ALERT === cdmaServiceCategory;
}
```


You made my day :dancers:


Added description in https://github.com/bboreham/weave/commit/0c98c4be86970ed60ede0ad358c3ad7eb9379ea5


sounds good for me. Just please ensure that for debugging purpose you can run it yourself


and ideally it should have really all dependencies needed for ci, so jenkins worker really need to have updated only one package.


@oritwas there's a typo in the description line. Also, can you add a reference in the commit message body to the commit the broke this?


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14073/testReport/) for this commit**


Does this break /say via IRC?


Closing a channel which other goroutines could be about to send on is risky.


I compiled Kobo version, I see the credits menu but when I open it is empty.


Exactly! And use proper formatting too, e.g. space beteween `)` and `{`.


LGTM.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14074/testReport/) for this commit**


e.g. http://www.bashbooster.net


@oritwas also, note that you should add an empty line after the description line to separate the body (doesn't show up on github, but it messes up other clients).


:+1: :)


No, i'm not the one in Bern :) I'll share on Skype later ;)


> I prefer not to change the global value of git-link-use-commit in the event I need to use the original git-link
> function behavior.

A similar -but slightly weaker argument- could be made for `git-link-open-in-browser`. But is I a good idea to create `git-link-force-open-in-browser`? Or wrap any config vars that come along in function that set them to `t`?

My biggest concern with `git-link-force-hash` (or similar functions) is that it may be temporary as, IMHO, it's trying to compensate for shortcomings of the UI. For example, if `git-link` was refactored to use various prefix args to control its behavior, would it be necessary?

Maybe `git-link-use-commit` should be buffer local? Under what circumstances would use use the commit hash over the branch name?


@oritwas otherwise looks fine!


Good catch! Fixed in f79973a54b2606f66a336b40e2f5495a033084f9


We could just name these `setUpDescription` and `setUpMembers`


TeamCity ChameleonForms :: Continuous Integration [Build 2.0.0+014](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3576&buildTypeId=bt20) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 3001 Build time: 0:0:1
### Failed tests
```
ChameleonForms.AcceptanceTests.dll: ChameleonForms.AcceptanceTests.ModelBinding.ModelBindingShould.Function_correctly_with_checkbox_and_radio_lists: Test(s) failed. TestStack.Seleno.Configuration.Interceptors.SelenoReceivedException : {"errorMessage":"Unable to find element with link text 'Model Binding Example with lists'","request":{"headers":{"Accept":"application/json, image/png","Connection":"Close","Content-Length":"64","Content-Type":"application/json;charset=utf-8","Host":"localhost:56049"},"httpVersion":"1.1","method":"POST","post":"{\"using\":\"link text\",\"value\":\"Model Binding Example with lists\"}","url":"/element","urlParsed":{"anchor":"","query":"","file":"element","directory":"/","path":"/element","relative":"/element","port":"","host":"","password":"","user":"","userInfo":"","authority":"","protocol":"","source":"/element","queryKey":{},"chunks":["element"]},"urlOriginal":"/session/0e9a6070-e830-11e4-a894-0730a5077d8c/element"}}
  ----> OpenQA.Selenium.NoSuchElementException : {"errorMessage":"Unable to find element with link text 'Model Binding Example with lists'","request":{"headers":{"Accept":"application/json, image/png","Connection":"Close","Content-Length":"64","Content-Type":"application/json;charset=utf-8","Host":"localhost:56049"},"httpVersion":"1.1","method":"POST","post":"{\"using\":\"link text\",\"value\":\"Model Binding Example with lists\"}","url":"/element","urlParsed":{"anchor":"","query":"","file":"element","directory":"/","path":"/element","relative":"/element","port":"","host":"","password":"","user":"","userInfo":"","authority":"","protocol":"","source":"/element","queryKey":{},"chunks":["element"]},"urlOriginal":"/session/0e9a6070-e830-11e4-a894-0730a5077d8c/element"}}
   at TestStack.Seleno.Configuration.Interceptors.CameraProxyInterceptor.Intercept(IInvocation invocation) in c:\ConsoleBuildAgent\work\f035b8dd08e41a61\src\TestStack.Seleno\Configuration\Interceptors\CameraProxyInterceptor.cs:line 52

ChameleonForms.AcceptanceTests.dll: ChameleonForms.AcceptanceTests.ModelBinding.ModelBindingShould.Function_correctly_with_default_form: Test(s) failed. TestStack.Seleno.Configuration.Interceptors.SelenoReceivedException : {"errorMessage":"Unable to find element with link text 'Model Binding Example'","request":{"headers":{"Accept":"application/json, image/png","Connection":"Close","Content-Length":"53","Content-Type":"application/json;charset=utf-8","Host":"localhost:56049"},"httpVersion":"1.1","method":"POST","post":"{\"using\":\"link text\",\"value\":\"Model Binding Example\"}","url":"/element","urlParsed":{"anchor":"","query":"","file":"element","directory":"/","path":"/element","relative":"/element","port":"","host":"","password":"","user":"","userInfo":"","authority":"","protocol":"","source":"/element","queryKey":{},"chunks":["element"]},"urlOriginal":"/session/0e9a6070-e830-11e4-a894-0730a5077d8c/element"}}
  ----> OpenQA.Selenium.NoSuchElementException : {"errorMessage":"Unable to find element with link text 'Model Binding Example'","request":{"headers":{"Accept":"application/json, image/png","Connection":"Close","Content-Length":"53","Content-Type":"application/json;charset=utf-8","Host":"localhost:56049"},"httpVersion":"1.1","method":"POST","post":"{\"using\":\"link text\",\"value\":\"Model Binding Example\"}","url":"/element","urlParsed":{"anchor":"","query":"","file":"element","directory":"/","path":"/element","relative":"/element","port":"","host":"","password":"","user":"","userInfo":"","authority":"","protocol":"","source":"/element","queryKey":{},"chunks":["element"]},"urlOriginal":"/session/0e9a6070-e830-11e4-a894-0730a5077d8c/element"}}
   at TestStack.Seleno.Configuration.Interceptors.CameraProxyInterceptor.Intercept(IInvocation invocation) in c:\ConsoleBuildAgent\work\f035b8dd08e41a61\src\TestStack.Seleno\Configuration\Interceptors\CameraProxyInterceptor.cs:line 52

```


Ah, yes. This should be `bindAddrs`. I'll fix it.


@eerac


Thanks. I will take care of it


@jeantil
Indeed `Predef.configuration` is part of Gatlinq's public API, I do no deny that.
What I was saying is that Gatling users are not meant to use that field, as its very existence is to tie the knots and pass the configuration where it is required to allow classes to be compiled.
If I may, let me take another example to make my point :

Java allows you to catch `Error`s and `OutOfMemoryError` is part of the JDK's public API. but should you catch `OutOfMemoryError` ?
The point is that is it not because you can, that you should :)

About configuration files: Indeed, using Typesafe Config is a good idea, with all the nice features it offers.
But, TBH, I'm really not sure that you should tie your custom configuration to gatling.conf instead of relying on dedicated config files.


Hello @arvidn (thanks for this amazing library). This const happens to be used inside disk_io_thread even if the TORRENT_USE_ASSERTS is not defined, then a link error (at leas in windows). Moving it outside the #if solved the problem for me.


Neither do I


每


但这不是


用户


I felt it was long, when qualifying a function parameter. I am ok either way. Will change it.


shall be a date


This was @charliesome's work ... my idea was just to use a database like gdbm which would be so much saner

https://github.com/SamSaffron/mime-types/commit/0cf4b1b145296c8878fe86405c7775df42ec8e4f


只要


I always need to use the commit hash for links to code in master branch because the master branch is not static; it changes with each commit.

The use of links with commit hash is in places like these: http://emacs.stackexchange.com/questions/10804/how-to-detect-if-current-major-mode-supports-which-function-and-semantic-find/10806#comment16708_10806

I basically need to get a 'permalink' to a code snippet so that I don't need to worry about the line reference changing each time the `master` updates.


I was thinking about a date and write String hahahahahahahaha


So I think that you should be returning a copy here (`new HashMap(processStatus)`).


It's probably time to start using _getArgsAsStr in the reboot command.


@jahto: yeah, last year I looked for buying a used VW T4. But still on my wish list. :-)
I'm not good at hiding, I even have a FB account with the same photo. ;-)


Confirmed for me, footer variable has to be initialized as empty array, then if the cachePath isn't set, one can return.


Or this should be a `ConcurrentHashMap`.


Why the `typeof`? This can be a `if(process.versions["node-webkit"])`


这句比较难懂，再看看如何分解这句话吧。


Век живи - век учись (c)


It's not clear to me that allowing them to send is any better.  Many of the senders will wait for a reply, no?  In the context of a test, it sounds nicer to know immediately that there is an attempt to use a stopped allocator than to have the request hang.


一个更好的模式是有一个列表应程序替代询问我们特殊权限的应用程序窗口。


I think you can compute `empty` too. Isn't it just `front == back`?


if you want the i size_t no cast needed


Looks good


Added this to avoid computing unnecessary derivatives. It recreates the math described at the top of http://docs.opencv.org/modules/calib3d/doc/camera_calibration_and_3d_reconstruction.html. Couldn't figure out a way to not have opencv compute derivatives (although I believe this can be done in C)


i want to update with the changes made.


pykickstart doesn't know anything about addons.  It's completely an anaconda concept.  pykickstart just provides the framework for defining new sections.  I think it's probably more correct to have handle-sshpw add the NullSection to its ksparser.